Skip to content

Conversation

@BMagnu
Copy link
Member

@BMagnu BMagnu commented Jan 22, 2026

Rolls back the insignia fix bit of #6849.
Apparently, two or so FSPort retail models had broken POFs that caused problems with the fix, so roll back the insignias to being incorrectly lit and faded for now, until someone has time to work around the broken models.

To fix the insignia bugs properly, one needs to revert this revert again, but make sure to analyze the insignia geometry in modelread.cpp, splitting non-adjacent faces so that multiple insignias in a single slot are correctly split into multiple insignia instances.

@BMagnu BMagnu added the regression A bug introduced by a new feature that breaks something which used to work. label Jan 22, 2026
@wookieejedi wookieejedi added this to the Release 25.0 milestone Jan 22, 2026
Copy link
Member

@wookieejedi wookieejedi left a comment

Choose a reason for hiding this comment

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

Revision matches look good, thanks for taking the time to work on these tedious clang appeasements.

@wookieejedi wookieejedi merged commit 83f0829 into scp-fs2open:master Jan 23, 2026
30 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

regression A bug introduced by a new feature that breaks something which used to work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants