Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Helmet HUDs skip over HUDs that are already in effect on the … #5655

Closed
wants to merge 1 commit into from

Conversation

Birdtalon
Copy link
Contributor

@Birdtalon Birdtalon commented Feb 3, 2024

…player (#5600)"

This reverts commit 56c7062.

#5600 has broken all helmet HUDs so this is going on TM until it can be fixed so marines who need huds can play.
[22:38:04]RUNTIME: Cannot read null.hudusers - code/modules/clothing/head/helmet.dm@697 (500 total)

--
Edit:

More detail this only affects the welding helmet visor not ALL visors.

@Birdtalon Birdtalon added the Test Merge Only This PR's purpose is for testmerges only. label Feb 3, 2024
@Birdtalon Birdtalon closed this Feb 4, 2024
github-merge-queue bot pushed a commit that referenced this pull request Feb 4, 2024
# About the pull request

So I am not able to reproduce the runtime spam errors reported in #5656
prompting #5655 but I have added null conditional operators where it is
complaining.

# Explain why it's good for the game

Less runtimes? Fixes #5656

# Testing Photographs and Procedure
Untested other than confirming it still seems fine? Please let me know
if you know how to reproduce it.

# Changelog
:cl: Drathek
fix: Added more null testing to helmet visors.
/:cl:
@Birdtalon Birdtalon deleted the huds branch February 25, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Test Merge Only This PR's purpose is for testmerges only.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant