Skip to content

Conversation

@Skyaero42
Copy link

Cleaned up unnecessary defines in language.h:

  • Removed multiple unused defines.
  • Inlined defines that were only referenced once. Eliminated unnecessary dependencies on language.h where possible.
  • Retained the GameArrayEnd define, as it’s used in multiple places and is relatively complex.

Already replicated in Generals.

Cleaned up unnecessary defines in `language.h`:

- Removed multiple unused defines.
- Inlined defines that were only referenced once to eliminate unnecessary dependencies on `language.h`.
- Retained the GameArrayEnd define, as it’s used in multiple places and is relatively complex.
@Skyaero42 Skyaero42 self-assigned this Oct 24, 2025
@Skyaero42 Skyaero42 added the Refactor Edits the code with insignificant behavior changes, is never user facing label Oct 24, 2025
Copy link

@xezon xezon left a comment

Choose a reason for hiding this comment

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

Nice.

@xezon xezon added the Approved Pull Request was approved label Oct 24, 2025
@xezon xezon changed the title refactor(language.h): remove unused and redundant defines refactor: Remove unused and superfluous defines from Language.h Oct 24, 2025
@xezon xezon changed the title refactor: Remove unused and superfluous defines from Language.h refactor: Remove superfluous defines from Language.h Oct 24, 2025
@xezon xezon merged commit 98e4f7e into TheSuperHackers:main Oct 24, 2025
19 checks passed
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Pull Request was approved Refactor Edits the code with insignificant behavior changes, is never user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants