mirror of
https://github.com/Cesarus85/ringfall-mjolnir-vr.git
synced 2026-10-25 14:00:28 +01:00
Jonas VolmanandClaude Opus 5
978fd91b7e
Split the camo regrowth guard's three-conjunct condition into nested ifs with explicit else arms assigning the default rate, and scope the set flag to the branch that uses it. Behaviour-identical: game_engine_running is still evaluated exactly once, the short-circuit order is unchanged, and every path that previously left set FALSE now assigns 1.f/120.f. That flips VC7 from eager to lazy placement of the partially redundant [ebp+8] reload. January duplicates it per predecessor -- 0x49c on the main path, 0x4f6 at the bypass merge, 0x533 in the else arm, with the jne going to 0x4f9 to skip the merge copy -- where we had hoisted a single dominating copy to 0x48a. _unit_update is now 1642 instructions with normalized_sha256 EQUAL to January: zero differing bytes, from 632. section_infos_equal still reports False on 2 of 259 relocations, both the same function-local static spelled two ways: build/split has `unit_update'::`114'::last_time where CL emits ?last_time@?HC@??unit_update@@9@9. That is the known naming gap in config/symbols.json, not a codegen difference -- all 259 addresses and types match and the other 69 differing records are forgiven by symbolic_target. CL /Zs /W3 unchanged at 43 diagnostics. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>