Skip to content

fix: Wrong usage of sizeof(this) in BitFlags::xfer(), W3DMPO::glueEnforcer() #1947

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.

Sign up for GitHub

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

Jump to bottom
Merged
xezon merged 5 commits into from
Dec 7, 2025
Merged

fix: Wrong usage of sizeof(this) in BitFlags::xfer(), W3DMPO::glueEnforcer() #1947

xezon merged 5 commits into from
Dec 7, 2025

Conversation

@bobtista
Copy link

@bobtista bobtista commented Dec 5, 2025 •
edited
Loading

1 Core/Libraries/Source/WWVegas/WWLib/always.h:147

  • Affects ~500+ classes using this macro for memory pool allocation.
  • Return value unused, and CI replays passed, so it seems safe.

2. BitFlags CRC Calculation (BitFlagsIO.h)

  • CRC calculation used sizeof(this), causing incomplete CRC hashing.
  • Behind retail compat flag

@Caball009
Copy link

Caball009 commented Dec 5, 2025 •
edited
Loading

This function serves no run-time purpose, so it cannot should not affect the replays in any way :)

@bobtista bobtista changed the title bugfix: fix sizeof(this) in W3DMPO_GLUE macro bugfix: fix sizeof(this) bugs found by clang-tidy Dec 5, 2025
@bobtista bobtista self-assigned this Dec 5, 2025
@bobtista bobtista marked this pull request as ready for review December 5, 2025 18:33
Skyaero42
Skyaero42 requested changes Dec 5, 2025
View reviewed changes
Caball009
Caball009 reviewed Dec 5, 2025
View reviewed changes
@bobtista bobtista force-pushed the bobtista/build/clang-tidy-sizeof-fixes branch from 027fb74 to 0e6a476 Compare December 6, 2025 02:13
xezon
xezon previously approved these changes Dec 6, 2025
View reviewed changes
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.

Looks very good.

@xezon xezon requested a review from Skyaero42 December 6, 2025 11:34
@xezon xezon added Refactor Edits the code with insignificant behavior changes, is never user facing Fix Is fixing something, but is not user facing Gen Relates to Generals ZH Relates to Zero Hour Minor Severity: Minor < Major < Critical < Blocker labels Dec 6, 2025
xezon
xezon reviewed Dec 6, 2025
View reviewed changes
@xezon xezon dismissed their stale review December 6, 2025 11:41

Mistake

@xezon xezon changed the title bugfix: fix sizeof(this) bugs found by clang-tidy bugfix: Fix wrong usage of sizeof(this) in BitFlags::xfer(), W3DMPO::glueEnforcer() Dec 7, 2025
@xezon xezon changed the title bugfix: Fix wrong usage of sizeof(this) in BitFlags::xfer(), W3DMPO::glueEnforcer() fix: Wrong usage of sizeof(this) in BitFlags::xfer(), W3DMPO::glueEnforcer() Dec 7, 2025
@xezon xezon merged commit 2b1bcee into TheSuperHackers:main Dec 7, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Caball009 Caball009 Caball009 left review comments

@xezon xezon xezon approved these changes

@Skyaero42 Skyaero42 Skyaero42 approved these changes

Assignees

@bobtista bobtista

Labels

Fix Is fixing something, but is not user facing Gen Relates to Generals Minor Severity: Minor < Major < Critical < Blocker Refactor Edits the code with insignificant behavior changes, is never user facing ZH Relates to Zero Hour

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bobtista @Caball009 @xezon @Skyaero42