mirror of
https://github.com/Druthulu/BFM-decomp
synced 2026-09-27 05:56:00 -04:00
feat(phase-29): T49 — func_80135260 x4 siblings + the 3,744-site decl conform
The §53 carve path banked only 4 of 137 siblings, ALL of them SC07 — the exact signature func_80177DA8 showed before its §99 fix, so the same lever applies. - jtbl_family_bank func_80135260: 4 BANKED / 133 gate-fail. The 4 are SC07 overlays. - conform_decls dry run confirmed the class: byte-true def is `s32 func_80135260(s32, s32, s16 *, s16 *)` but 3,744 declaration sites say `(s32, s32, s32, s32)` — params 3 and 4 declared s32 where the byte truth is s16 *. Return type agrees, so the §85 return-axis precondition does not fire. - Applied: 3,744 sites rewritten across 2,021 files; the tool's R32 assertion reports "non-canonical declarations remaining: 0 OK (axis complete)". It correctly SKIPPED the 5 DEFINING TUs (the 4 SC07 banks + ov_SC01_077) — a defining TU owns its own declarations, since per-overlay byte-true signatures legitimately differ under §16 loose typing. - This touches src/shared/engine_core.h, so it is FLEET-SHARED and R22 was mandatory (§61/§63): R22 clean-fleet 140 passed, 0 failed of 140. Also recorded: jtbl_family_bank's [gather_externs] warning named func_80135D20 as an undeclared referenced symbol, but that symbol appears ONLY in the draft's header COMMENTS (lines 3 and 29) — a comment-scanning false positive, same class as the Phase-19 gen_harvest_targets garbled-hint bug. It was not the cause of the 133 failures. Next: re-run the carve path for the remaining 133 siblings now that the decl axis is conformed.
This commit is contained in:
@@ -26239,7 +26239,7 @@
|
||||
|
||||
#define DEFINE_func_801571C4() \
|
||||
extern s32 func_80135004(s32 a0, void *a1, s32 a2); \
|
||||
extern s32 func_80135260(s32 a0, s32 a1, s32 a2, s32 a3); \
|
||||
extern s32 func_80135260(s32 arg0, s32 arg1, s16 *arg2, s16 *arg3); \
|
||||
extern s32 func_8014C918(s32 a0, s32 a1); \
|
||||
extern void func_8014C4AC(s32 a0, s32 a1, s32 a2, s16 *a3, s32 a4); \
|
||||
extern u8 D_801202A0[]; \
|
||||
@@ -27023,7 +27023,7 @@
|
||||
#define DEFINE_func_8014A048() \
|
||||
extern u8 func_8014BEF8(void); \
|
||||
extern void func_8012F14C(s32); \
|
||||
extern s32 func_80135260(s32, s32, s32, s32); \
|
||||
extern s32 func_80135260(s32 arg0, s32 arg1, s16 *arg2, s16 *arg3); \
|
||||
extern void func_8014A1B0(s32 a0, s32 a1); \
|
||||
extern u8 D_801202A0[]; \
|
||||
s32 func_8014A048(s32 param_1) { \
|
||||
|
||||
Reference in New Issue
Block a user