Skip to content

Conversation

xezon
Copy link

@xezon xezon commented Sep 10, 2025

This change fixes a few minor miscellaneous memory issues I spotted when removing the superfluous null tests. I suspect these things cause no actual runtime issues.

TODO

  • Replicate in Generals

@xezon xezon added this to the Stability fixes milestone Sep 10, 2025
@xezon xezon added Minor Severity: Minor < Major < Critical < Blocker Memory Is memory related Fix Is fixing something, but is not user facing labels Sep 10, 2025
Copy link

@roossienb roossienb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@xezon xezon force-pushed the xezon/fix-memory-management branch from 5a43092 to 2f8ba88 Compare September 12, 2025 07:12
@xezon
Copy link
Author

xezon commented Sep 12, 2025

Replicated in Generals without conflicts.

@xezon xezon merged commit 4f06c9b into TheSuperHackers:main Sep 12, 2025
18 checks passed
@xezon xezon deleted the xezon/fix-memory-management branch September 12, 2025 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Is fixing something, but is not user facing Memory Is memory related Minor Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants