Skip to content

menu_compa: 95.86% -> 96.41% - #17672

Open
SirEnkido wants to merge 2 commits into
zcanann:mainfrom
SirEnkido:agent/menu_compa
Open

menu_compa: 95.86% -> 96.41%#17672
SirEnkido wants to merge 2 commits into
zcanann:mainfrom
SirEnkido:agent/menu_compa

Conversation

@SirEnkido

Copy link
Copy Markdown

What changed

Two source-shape refinements inside CMenuPcs::CompaDraw (src/menu_compa.cpp):

  • the food-icon rows loop computes its y offset inline as i * 0x28 (strength-reduction form the target uses) instead of a separate accumulator, and the +1 food-table index is folded into the array subscript so it lands in the load displacement
  • the fill-width local is hoisted to function scope and reused for the remaining-width computation, pinning the value in f29 exactly as the target does (variable reuse across roles is a recurring pattern in this codebase's matched functions — see WORK_SPLIT.md "whole-function register webs")

Evidence

  • main/menu_compa fuzzy (report.json): 95.86% → 96.41% (96.40594)
  • DOL verified byte-identical (sha1 matches config/GCCP01/build.sha1)
  • Rebuilt in a clean worktree and independently re-verified before submission

Notes for review

  • Reusing fillW for the remaining-width value trades the clearer name remainW for the f29 pin; stated openly rather than hidden.

Produced by Claude agents following AGENTS.md / WORK_SPLIT.md, operated by @SirEnkido.

🤖 Generated with Claude Code

SirEnkido and others added 2 commits July 17, 2026 21:21
…uction; food index +1 folded into displacement)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… for remain-width, pins f29 like target)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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