Skip to content

mes: 94.81% -> 94.89% - #17673

Open
SirEnkido wants to merge 3 commits into
zcanann:mainfrom
SirEnkido:agent/mes
Open

mes: 94.81% -> 94.89%#17673
SirEnkido wants to merge 3 commits into
zcanann:mainfrom
SirEnkido:agent/mes

Conversation

@SirEnkido

Copy link
Copy Markdown

What changed

CMes::addString (src/mes.cpp):

  • scoped #pragma opt_lifetimes off around the function (off/reset paired, house-accepted pattern)
  • the next message byte is read via a direct unsigned char pointer dereference instead of the single-use ReadMesChar inline helper — this drops a return-temp mr the target does not have
  • follow-up commit removes the now-unused ReadMesChar helper (no other call sites; score unchanged)

Evidence

  • main/mes fuzzy (report.json): 94.8078% → 94.8926%
  • All three intermediate scores in the commit messages were independently reproduced by rebuilding each revision
  • build/GCCP01/ok passes

Notes for review

  • If the original source used ReadMesChar elsewhere in future matches, it can be trivially re-added; today it has no other call sites.
  • Per WORK_SPLIT.md guidance, the scoped pragma should be retested for removal after future structural changes to this function.

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

🤖 Generated with Claude Code

SirEnkido and others added 3 commits July 17, 2026 21:02
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…r deref, drops return-temp mr)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…94.8926% unchanged)

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