Skip to content

System/MarNameRefGen_BossEnemy: construct Boss Pakkun variants - #199

Closed
KakarottoCake wants to merge 1 commit into
doldecomp:mainfrom
KakarottoCake:decomp-system
Closed

KakarottoCake wants to merge 1 commit into
doldecomp:mainfrom
KakarottoCake:decomp-system

Conversation

@KakarottoCake

Copy link
Copy Markdown
Contributor

Restores four Boss Pakkun factory cases in TMarNameRefGen::getNameRef_BossEnemy: BossPakkun, KBossPakkun, and their managers.

MarNameRefGen_BossEnemy fuzzy match: 25.19% -> 30.63%. No new exact functions; the unit still has missing cases.

Checked with native Windows bare Ninja, clang-format, review-lint, and changed-symbol-order against the baseline object (0 new symbol errors).

@decomp-dev

decomp-dev Bot commented Sep 27, 2026

Copy link
Copy Markdown

Report for GMSJ01 (7846008 - defd5a3)

📈 2 improvements in unmatched items
Unit Item Bytes Before After
mario/System/MarNameRefGen_BossEnemy TMarNameRefGen::getNameRef_BossEnemy(const char*) const +205 35.50% 43.18%
mario/System/MarNameRefGen_BossEnemy .rodata +170 45.42% 55.39%

Report for GMSP01 (7846008 - defd5a3)

📈 2 improvements in unmatched items
Unit Item Bytes Before After
mario/System/MarNameRefGen_BossEnemy TMarNameRefGen::getNameRef_BossEnemy(const char*) const +213 40.64% 49.74%
mario/System/MarNameRefGen_BossEnemy .rodata +163 45.87% 56.71%

@KakarottoCake

Copy link
Copy Markdown
Contributor Author

Closing this PR because its scope is too narrow. Reworking the full Mario/System directory in a replacement PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant