Eleventh cleanup sweep. Five asm files and their includes removed:
asm/main_0205B6CC.s sub_0205B6CC
asm/main_0202C75C.s sub_0202C75C
asm/overlay_29_0232CDA4.s DoMoveOneShot
asm/overlay_29_0232A078.s DoMoveNightShade
asm/overlay_29_023016D8.s DisplayRunAwayIfTriggered
All five merged into an adjacent source rather than splitting one. None carries
a region conditional, so US alone verifies this commit.
sub_0205B6CC is a 64-bit unsigned >= 2 (cmp/cmpeq/movhs) on a value at offset
0xC. A u64 member sits there: MWCC aligns long long to 4, not 8. Rebuilding the
value from two u32s scores 200, and `> 1` in place of `>= 2` scores 310.
sub_0202C75C uses struct unk_0202C5E0, the collection-menu window struct its
neighbours already use; GetWindowIdSelectedMenuItemIdx immediately before it
already spells the same `GetWindowContents(window_id) + 4`. field_0x1AC[6]
covered 0x1B0/0x1B1 and is split into three members.
DoMoveOneShot and DoMoveNightShade are twins staging eleven arguments for
CalcDamageFixedWrapper, which had no declaration in the tree. The only obstacle
was the temporary holding GetMoveType's result: as enum type_id, -enum min sizes
it to a byte and emits an and #0xff the target lacks (950); as s32, 0.
DisplayRunAwayIfTriggered needs Min(999, max_hp), not Min(max_hp, 999) -- Min is
a static inline, so the argument order is not free (200). That exact expression
in that exact order already sat in the NONMATCHING block of AiMovement in
dungeon_ai_movement.c; the guess was right and had never been confirmed.
include/main_02042AF8.h's provisional `s32 sub_0202C75C(s8 a);` is retired in
favour of the real header. It was written from the one call site, which passes an
s8 field; the parameter is a window id.
A sixth was prepared and not landed: FixedPoint64ToInt (asm/main_02001CB0.s)
stands at 225. The target keeps an arithmetically redundant & 0xFFFF0000, and
whether MWCC folds it away is decided by which side of the | the masked term is
written on -- masked second folds it (735), masked first keeps it (225). Writing
it first is also what makes it evaluate first, and the target needs it preserved
but evaluated second; ten spellings found none that does both. Spelling it as a
real 64-bit shift scores 1020.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Tenth cleanup sweep. Five asm files and their includes removed:
asm/overlay_29_0232CE7C.s DoMoveReviver
asm/overlay_29_02327940.s DoMoveRapidSpin
asm/overlay_29_022EFA74.s TryActivateIqBooster
asm/main_02027AA0.s sub_02027AA0
asm/main_020568DC.s sub_020568DC
All five merged into an adjacent source rather than splitting one. Four of the
five carry a region conditional, so all three ROMs were built; sub_020568DC is
the exception and US alone would have covered it.
TryActivateIqBooster sat at 60 with r6/r7/r8 rotated one place and every
differing row an r-marked register swap. Moving the IQ local's declaration ahead
of the other two takes it to 0. This contradicts the MATCHING_TIPS entry claiming
declaration order never moves callee-saved assignment; that measurement was on
SetLeaderAction's ~75 locals, where register pressure decides the colouring. On
four locals with no pressure, first-definition order is what is left.
DoMoveRapidSpin sets its return to 1 before the random-outcome call, so a failed
random check still returns TRUE. The early-return spelling scores 570; a result
variable set inside the damage branch scores 0.
sub_02027AA0's region difference is not an offset shift: JP writes slot 3 where
the others write slot 0 and vice versa, mirrored for the second case. The rodata
a few symbols away corroborates it -- _020AFD94 and _020AFD98 are defined in
swapped order under the same #ifdef.
sub_020568DC follows GetAppointedLeaderMemberIdx, which lives in the file it
merged into and is the same function specialised to the active team. The
(&t->number_active_team_members_main)[team] idiom is already used by three
existing files.
DoMoveReviver was on the stalled list from 4d2a484c but had never been attempted;
it was set aside for needing region-conditional C, which is one #define.
sub_020568DC and TryActivateIqBooster were likewise untried rather than stuck.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Confirmed by matching builds of all three ROMs: US, EU and JP.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Ninth cleanup sweep. Five asm files and their includes removed:
asm/overlay_29_0231E990.s ShouldTryEatItem
asm/overlay_29_0232D378.s DoMoveGuardSwap
asm/overlay_29_0232A490.s DoMoveSwallow
asm/overlay_29_02329674.s DoMoveHelpingHand
asm/main_0205B738.s sub_0205B738
All five merged into an adjacent source rather than splitting one.
Three declarations the tree already carried are contradicted by these bytes and
are corrected here, touching five files this sweep did not otherwise decompile:
SubstitutePlaceholderStringTags took `u8 *string_id` in four files and `int a` in
a fifth. DoMoveGuardSwap passes a literal 1, which no pointer parameter can take
without a cast; the existing call sites all passed 0, which is why nothing caught
it. The four are now s32, agreeing with the one that was already an integer.
TryIncreaseHp took `s16 hp` in two files. DoMoveSwallow loads its argument from a
word array and passes it untouched -- s16 inserts a sign extension and scores
210, s32 scores 0. The two existing call sites load s16 data with ldrsh, so the
value is already extended and the wider declaration is byte-neutral for them.
ShouldTryEatItem's parameter is signed: the range check emits blt/ble, while
enum item_id under -enum min compares unsigned and scores 400. s16 matches the
adjacent IsItemValidVeneer and every caller's ldrsh.
DoMoveSwallow also needed its two arms in the other source order, the same rule
as sub_02009DCC last sweep: MWCC lets the first arm written fall through.
sub_0205B738's copy at +0xB0 is element [1] of struct0, not a member: 0xB0 is
sizeof(struct unkStruct_020B0A54), as sub_0205B77C (a * 176 + ...) and
sub_0205B7C8 (MemsetSimple(..., 0xB0)) already say. Reading it as a member and
appending one scored 0 on a scratch but changed the struct's size, and with it
the stride in three neighbours and the copy length in a fourth -- 29 binaries
moved. No header change is needed; the array spelling also scores 0.
Three of the five log a message whose id differs between US and JP by no constant
offset, so each is a file-scope #ifdef JAPAN define, as dungeon_ai.c and
dungeon_projectile_throw.c already do.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Three carry region conditionals, so this was confirmed by matching builds of all
three ROMs: US, EU and JP.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Eighth cleanup sweep. Five asm files and their includes removed:
asm/overlay_29_0230F840.s TeamMemberHasExclusiveItemEffectActive
asm/overlay_29_023276E8.s DoMoveTickle
asm/overlay_29_0232C538.s DoMoveSwitchPositions
asm/overlay_29_022FF8C8.s GetMobilityTypeAfterIqSkills
asm/overlay_29_0233861C.s UpdateShouldBoostHiddenStairsSpawnChance
All five merged into an adjacent source rather than splitting one.
DoMoveTickle was the stall recorded on 78b52f9d at score 15. The target reaches
ATK_STAT_IDX through SPATK_STAT_IDX's pool entry at offset 4, which is what the
two symbols' adjacency in rodata makes them. Declaring SPATK_STAT_IDX with array
type and indexing [1] emits that; taking the address of a scalar declaration and
indexing does not. DoMoveMetalSound, which shares the file, moves to
SPATK_STAT_IDX[0] for the same declaration.
TeamMemberHasExclusiveItemEffectActive replaces the provisional
`bool8 TeamMemberHasExclusiveItemEffectActive(u8);` that src/overlay_29_02338548.c
declared from its call site. The parameter is enum exclusive_item_effect_id --
u8 scores 0 in isolation but the build's -W error rejects the implicit conversion
at the inner call to ExclusiveItemEffectIsActive__0230F8AC. Both call sites move
from 0x5e/0x5f to EXCLUSIVE_EFF_MORE_KECLEON_SHOPS/EXCLUSIVE_EFF_MORE_HIDDEN_STAIRS.
DoMoveSwitchPositions raises exp_yield to 1 when below it; spelled `== 0` rather
than `< 1` it scores 405.
GetMobilityTypeAfterIqSkills is a whole function inside #ifndef JAPAN, so the
guard moves into the .c and around the prototype in the .h.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Three carry region conditionals, so this was confirmed by matching builds of all
three ROMs: US, EU and JP.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Seventh cleanup sweep. Five asm files and their includes removed:
asm/overlay_29_0232E400.s DoMoveWakeUpSlap
asm/main_0204317C.s FreeUnkStorageStruct0x18c0
asm/main_02009DCC.s sub_02009DCC
asm/main_020018A4.s sub_020018A4
asm/main_0200C4CC.s sub_0200C4CC
All five merged into an adjacent source rather than splitting one, so the sweep
removes five files and adds none.
DoMoveWakeUpSlap doubles the damage multiplier when the target is in any of the
three sleep-class states, then ends the status. EndSleepClassStatus takes five
arguments here, not four: the target loads r2=0, r3=1 and pushes a third flag on
the stack.
sub_020018A4 rounds a 64-bit value up to the next multiple of 0x100 and returns
the high 56 bits. The `and r2, r1, #0` that opens it is the high half of a 64-bit
`x & 0xFF`, not a separate test.
sub_02009DCC clears one bit of a 5-bit mask per non-zero entry in five byte
arrays at stride 5, then merges it into DISPCNT for the requested engine. The
zero-engine arm has to come first in the source or the wrong arm is predicated.
sub_0200C4CC copies 232 bytes out of _02094C18, whose size is fixed by the gap to
_02094D00.
Field types forced by the access widths in main_02042AF8.h: struct unk_02043148's
field_0x0 becomes s32 and field_0x7D5 splits into two s8s plus padding; struct
unk_020AFEE0's field_0xC becomes s32, which was the last differing row in
FreeUnkStorageStruct0x18c0.
struct unk_02094C18 is new. Its member is u32[0x3A] rather than u8[0xE8] because
the copy's alignment, and so its instruction shape, comes from the member type.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Sixth cleanup sweep. Four asm files and their includes removed:
asm/overlay_29_02328A74.s DoMoveDamagePoison40
asm/overlay_29_02329FE0.s DoMoveMudSlap
asm/overlay_29_0232B820.s DoMoveMistBall
asm/main_0200F18C.s GetEquivItemIndex
The three move effects share DoMoveBubble's shape: deal damage, and on a random
outcome apply a secondary effect, returning whether damage landed rather than
whether the effect did. The result variable doubles as an argument in two of
them -- MistBall passes it as both the stage count and the checkProtected flag,
MudSlap passes 0 as the chance.
GetEquivItemIndex walks the 50 bag slots with a cursor, testing each item's
exists flag as `(flags & 1) != 0` so the boolify emits tst/movne/moveq, and
returns the index of the first equivalent item or -1.
A fifth was prepared and not landed: GetNextIqSkill (asm/main_02058F24.s) sits at
30, the whole difference being that the target keeps its skill-list cursor in lr
where the candidate uses r5. Four spellings tried -- hoisting the value local,
moving the cursor declaration, a group temporary, and a while loop -- all 30 or
worse.
Authored by Claude (Opus 5) under human direction. All four verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Fifth cleanup sweep; five more asm files and their includes removed:
asm/overlay_29_023491C4.s IsOutlawOrChallengeRequestFloor
asm/main_0202AAE8.s sub_0202AAE8
asm/main_0202BC60.s sub_0202BC60
asm/overlay_29_023261A0.s DoMoveBubble
asm/main_0205BCC4.s sub_0205BCC4
sub_0202AAE8 and sub_0202BC60 are the same function over different window
states: check unk_0202AAA8::field_0x19C against a set, then set two flags, clear
field_0x1A4, call sub_0202830C with -5 and advance the state. Both write a byte
at 0x1A1 that the struct covered only as padding after field_0x1A0, so that gap
is now named field_0x1A1 with field_0x1A2[2] holding the remainder -- same total
size.
Three spellings that mattered:
- IsOutlawOrChallengeRequestFloor is three ORed predicates and only matches with
each operand written `!= 0`; a plain || scores 300 and early returns 1300.
- DoMoveBubble returns whether damage was dealt, not whether the speed drop
landed, so the result is funnelled through one variable which is also passed
as LowerSpeed's stage count.
- sub_0205BCC4 must not hoist the destination pointer above its NULL check; a
local shared by both branches scores 830, one declared inside the else branch
matches.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Fourth cleanup sweep; five more asm files and their includes removed:
asm/overlay_29_0232B8E4.s DoMoveCalmMind
asm/overlay_29_0232D22C.s DoMoveHammerArm
asm/overlay_29_023495E8.s IsDestinationFloorWithFleeingOutlaw
asm/overlay_29_023492D4.s IsLegendaryChallengeFloor
asm/main_02030850.s CloseAdvancedTextBox2
The two mission predicates merged into files that already declared
IsCurrentMissionType and IsCurrentMissionTypeExact with enum mission_type
parameters, so they use those declarations and the MISSION_* enumerators rather
than fresh externs with integer types. IsLegendaryChallengeFloor's floor check
is a byte subtraction written `+ 0xFF`, the same encoding DoMoveGust needed.
Both predicates only match written as a nested `if` that falls through to a
single return; the early-return spellings score 610 and 405.
CloseAdvancedTextBox2 uses the existing sub_0202836C, sub_0202812C, DeleteWindow
and MemFree declarations; only sub_020288DC needed a provisional extern.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Third cleanup sweep; five more asm files and their includes removed:
asm/overlay_29_023021F0.s UpdateIqSkillsWrapper
asm/overlay_29_022EBC98.s SetActionUseMovePlayer
asm/overlay_29_02349658.s ov29_02349658
asm/main_0205EBF0.s sub_0205EBF0
asm/overlay_29_022F52B0.s ov29_022F52B0
SetActionUseMovePlayer sets ACTION_USE_MOVE_PLAYER and then two
action_parameters[].action_use_idx fields, using the enumerator and struct that
already exist. sub_0205EBF0 indexes mission_deliver_list::unk18, the struct
src/main_0205C73C.c already uses, rather than a fresh placeholder.
NOT done, deliberately: src/overlay_29_022F0EDC.c still carries the provisional
`extern s32 SetActionUseMovePlayer();`. Replacing it with an include of the real
header, which declares the function void, breaks the ROM -- SetLeaderAction's
match depends on the s32 form. The two declarations therefore still disagree.
That is a real defect in one of the two, not a formality, and it is left alone
rather than papered over; see the reasoning note.
ov29_022F52B0 is a bare veneer with no argument setup, so its arity is not
evidenced by anything and void(void) is a guess.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Second cleanup sweep. Each was the only function left in its own asm file, so
each removes an asm file and its include:
asm/overlay_29_0234B0CC.s SetMessageLogPreprocessorArgsStringToName
asm/overlay_29_023268CC.s DoMoveSmokescreen
asm/overlay_29_02325F74.s DoMoveMorningSun
asm/overlay_29_02326E20.s DoMoveGust
asm/overlay_29_022EA980.s GenerateDungeonRngSeed
Two needed a second reading:
- DoMoveGust doubles the damage multiplier when the defender's
bide_class_status.bide is 7 or 8. The target tests it as a byte with
`add #0xF9`, so the source adds 0xF9 rather than subtracting 7; `- 7` compiles
to `sub #7` and does not match.
- GenerateDungeonRngSeed runs the LCG twice, stores the second result back to
prng_state::preseed, and returns the two halves recombined. The operand order
in the final or-expression decides whether the mask or the shift is scheduled
first; taking the shifted half first matches.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Each of these was the only function left in its own asm file -- fragments left
behind when earlier work split a larger file around them. Decompiling them
removes five asm files and their includes outright:
asm/main_0204FD38.s IncrementNbSteals
asm/overlay_29_022EAC78.s ov29_022EAC78
asm/overlay_29_0234B010.s ov29_0234B010
asm/main_0202BCDC.s IsAdvancedMenuActive
asm/main_02030A2C.s SetWindowTitle
IncrementNbSteals and ov29_022EAC78 are empty in retail -- a single bx lr. Their
signatures are not evidenced by anything and are written void(void).
IsAdvancedMenuActive reads unk_0202AAA8::field_0x1A0, the same field and the same
shape as CheckParentMenuField0x1A0 next door.
SetWindowTitle writes a halfword at offset 0x20 of the same struct, which was
inside the opaque field_0x9[0x97] placeholder. That array is split into
field_0x9[0x17] / u16 field_0x20 / field_0x22[0x7E] -- same total size, and
nothing indexed the placeholder.
Authored by Claude (Opus 5) under human direction. All five verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
GetLeaderMonster, SetDungeonRngPreseed, GetDungeonRngPreseed,
SetDungeonRngPreseed23Bit, FindClosestUnoccupiedTileWithin2,
FindFarthestUnoccupiedTileWithin2, FindUnoccupiedTileWithin3,
FixedRoomActionParamToDirection, ov29_022E9FC0 and ov29_022E9FD0.
The three preseed accessors read and write prng_state::preseed at offset 8,
using the struct dg_random.h already defines. SetDungeonRngPreseed23Bit sets bit
0 and masks the top byte, which is where its name comes from.
GetLeaderMonster merged into dg_uty.h alongside GetLeader, so the provisional
`extern struct monster *GetLeaderMonster();` that src/overlay_29_022F0EDC.c
carried is removed -- that file already includes dg_uty.h, which now declares it
properly with a (void) parameter list.
The three FindNearbyUnoccupiedTile wrappers pass a displacement table and a
count (0x19 within 2, 0x31 within 3). That helper and its tables are undeclared
in the tree, so their prototypes are written from these call sites and are
provisional.
Authored by Claude (Opus 5) under human direction. All ten verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
SetMessageLogGroupStartFlag, GetMessageLogPreprocessorArgs,
InitMessageLogPreprocessorArgs, SetMessageLogPreprocessorArgsFlagVal,
SetPreprocessorArgsIdVal, SetMessageLogPreprocessorArgsNumberVal,
SetMessageLogPreprocessorArgsString, SetMessageLogPreprocessorArgsSpeakerId,
SetMessageLogPreprocessorArgsSpeakerId0x30000 and ov29_0234AFFC.
All reach a preprocessor-argument block through MESSAGE_LOG_INFO, a 12-byte
global holding a flag byte and a pointer. The block is the existing
struct preprocessor_args from main_0200C4FC.h: its member offsets -- id_vals at
+0x10, number_vals at +0x24, strings at +0x38, speaker_id at +0x4C -- land
exactly on the asm's 0xCAC, 0xCC0, 0xCD4 and 0xCE8 relative to the 0xC9C base.
Ten independent accessors agreeing with that layout is corroboration of the
existing struct rather than a new claim.
The global itself gets a placeholder, struct unk_023537CC, named for its
address; only its flag byte and pointer are known.
Authored by Claude (Opus 5) under human direction. All ten verified at score 0
(eleven candidates matched; ten landed). Confirmed by a matching build:
build/pmdsky.us/pmdsky.us.nds: OK. None carries a region conditional, so US
alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The previous commit dropped eight functions whose only difference from the
target was instruction order: retail loads the stat global before materialising
the constant destined for the stack, the candidate after. The cause is aliasing.
Declared as plain externs, MWCC must assume the stores to [sp] can affect the
global, so the load cannot move above them. Declaring the globals const frees
it, and all eight match:
DoMoveGrowl, DoMoveLowerDefense1, DoMoveFeatherDance, DoMoveFakeTears,
DoMoveMetalSound, DoMoveScreech, DoMoveCharm and DoMoveHealOrder. Also
LogMessageQuiet and LogMessageWithPopup, which pass FALSE/TRUE to LogMessage.
Every existing `extern struct StatIndex` in src/ is const-ified to match, since
the globals cannot be const in one translation unit and not another. Thirteen
files touched; the functions already landed against them still match, which is
independent confirmation of the reading.
DoMoveMetalSound lowers by 3 stages, not the 2 assumed when it was first
written; the asm says `mov r3, #3`.
Authored by Claude (Opus 5) under human direction. All ten verified at score 0.
Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a
region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Landed as one batch after verifying every function individually, rather than in
groups of ten. Five clusters:
- Sound and effect veneers that tail-call a shared helper with fixed ids:
PlaySeByIdIfShouldDisplayEntity, PlayEffectAnimationPos,
PlayEffectAnimationPixelPos, ChangeMonsterAnimation and ov29_022E67E4.
- Four DoMoveDamage__* wrappers plus DoMoveVacuumCut, all
`return DealDamage(...) != 0`.
- DoMoveExpose, DoMoveSport, DoMoveMirrorMove, DoMoveMiracleEye,
DoMoveNightmare, DoMoveSureShot, DoMoveFeint and DoMoveRockPolish, which
branch or sequence two calls rather than forwarding once.
- Minimap byte accessors over GetMinimapData, and GetTileTerrain.
- Wrappers whose argument setup evidences the callee arity:
ShouldDisplayEntityWrapper, UseSingleUseItemSelf, CalcSpeedStageWrapper,
GenerateCleanItem and three ov29_* forwarders.
Eight further candidates were prepared and are deliberately NOT included:
DoMoveHealOrder, DoMoveScreech, DoMoveCharm, DoMoveGrowl, DoMoveLowerDefense1,
DoMoveFakeTears, DoMoveFeatherDance and DoMoveMetalSound. Each passes a constant
on the stack to a six-argument stat helper, and each differs from the target only
in instruction *order*: retail loads the stat global before materialising the
constant, the candidate after. Same instructions, same count. They stay in asm.
Authored by Claude (Opus 5) under human direction. All 42 verified at score 0
individually before landing; confirmed together by a single matching build:
build/pmdsky.us/pmdsky.us.nds: OK. None carries a region conditional, so US
alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveLowerAccuracy1, DoMoveSweetScent, DoMoveNastyPlot, DoMoveSwordsDance,
DoMoveTailGlow, DoMoveBlowback, DoMoveHiddenPower, DoMoveDecoy,
DoMovePsychoShift and DebugRecruitingEnabled.
Five more stat handlers use the StatIndex helpers from move_orb_effects.h. The
rest depart from the template:
- DoMoveBlowback passes the attacker's own action.direction, read through
GetEntInfo.
- DoMoveHiddenPower forwards move and item_id to DealDamage with a 0x100
multiplier, its fifth argument on the stack.
- DoMoveDecoy passes three constants, the last on the stack.
- DoMovePsychoShift guards on attacker != defender before transferring.
DebugRecruitingEnabled returns TRUE unconditionally; dungeon_recruitment.h
already declares it, so the signature is not a guess.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate. Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
None carries a region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveTag0x1AB, DoMoveDoubleTeam, DoMoveMinimize, DoMoveAmnesia,
DoMoveBoostAttack1, DoMoveBoostDefense1, DoMoveBoostDefense2, DoMoveDefenseCurl,
DoMoveGrowth and DoMoveFamish.
Eight forward to BoostOffensiveStat / BoostDefensiveStat / BoostHitChanceStat
with ATK_STAT_IDX or SPATK_STAT_IDX and a stage count. These take
struct StatIndex by value, per move_orb_effects.h, which is why the asm loads the
global's word into r2.
src/overlay_29_0232E250.c declared those globals as s32 and BoostDefensiveStat
with an (s32, s16) tail, both contradicting move_orb_effects.h. Since the two
declarations lived in different translation units the compiler never saw the
disagreement and the ROM still matched. Replaced with an include of the canonical
header; DoMoveDefendOrder is unchanged and still matches.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate once the canonical StatIndex signature was used. Confirmed by a
matching build: build/pmdsky.us/pmdsky.us.nds: OK. None carries a region
conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DungeonRngSetPrimary, DungeonRngSetSecondary and DungeonRngUnsetSecondary write
prng_state::use_secondary and ::idx_secondary, using the struct dg_random.h
already defines and the extern src/dg_random.c already uses. No new types.
PlayLevelUpSound, PlayDungeonTipSound__022EB63C, PlayDungeonTipSound__022EB66C,
ov29_022EAC9C and ov29_022EACAC each tail-call sub_02017C50 with a fixed id
(1, 7, 7, 0, 5). PlaySeByIdIfNotSilence and PlayMeByIdIfNot998 guard on a
sentinel before forwarding -- 0x3F00 and 998 respectively, the latter matching
the name.
sub_02017C74 keeps the signature src/overlay_25_init.c already declares for it.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate. Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
None carries a region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveHealBlock, DoMoveEmbargo, DoMoveGravity, DoMoveMagnetRise, DoMoveTrickRoom,
DoMovePowerTrick, DoMoveInvisify, DoMoveMetalBurst, DoMoveGrudge and
DoMoveDamageStealItem. This exhausts the forward-and-return-TRUE family.
Three depart from the template:
- DoMoveInvisify passes the attacker as both arguments (mov r1, r0), so
Invisify targets the user rather than the defender.
- DoMoveGrudge and DoMoveDamageStealItem are tail calls rather than
call-then-return-TRUE, so they are written `return f(...)`. DoMoveDamageStealItem
forwards its own four arguments unchanged to DoMoveTakeaway, which is still asm.
DoMoveMetalBurst passes 15 to SetReflectStatus, a third distinct value for that
selector after 4 (Counter) and 10 (Rebound).
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate. Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
None carries a region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveShocker, DoMoveOneRoom, DoMoveHurl, DoMoveMobile, DoMoveSeeStairs,
DoMoveLongToss, DoMovePierce, DoMoveAquaRing, DoMoveGastroAcid and
DoMoveLuckyChant -- the same forward-and-return-TRUE template as the previous
batches. All ten helpers are undeclared in the tree and take provisional externs
written from the call site.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate. Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
None carries a region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveRebound, DoMoveStayAway, DoMovePowerEars, DoMoveSlowDown,
DoMoveSearchlight, DoMovePetrify, DoMovePounce, DoMoveTrawl, DoMoveDrought and
DoMoveHpGauge.
Two of them give a second data point for a helper argument seen earlier:
SetReflectStatus takes 10 here where DoMoveCounter passes 4, and TryWarp takes 1
where DoMoveWarp passes 0. Both third arguments are therefore selectors of some
kind; nothing here establishes what they select.
DoMoveSlowDown forwards to LowerSpeed with the same arguments as
DoMoveLowerSpeed1. The remaining helpers are undeclared in the tree and take
provisional externs written from the call site.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate. Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
None carries a region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveWrap, DoMoveMagicCoat, DoMoveProtect, DoMoveDestinyBond, DoMoveMirrorCoat,
DoMoveSnatch, DoMoveReflect, DoMoveSeeTrap, DoMoveScan and DoMoveNoMove.
Nine forward to a two-argument helper and return TRUE; DoMoveNoMove passes a
third argument of 0. None of the ten helpers is declared in the tree, so each
takes a provisional extern written from its call site.
DoMoveSeeTrap and DoMoveScan forward to RevealTrapsNearby and RevealItems, the
first helpers in this family that are not TryInflict*Status.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate. Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
None carries a region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveLowerSpeed1, DoMoveParalyze__02328230, DoMoveParalyze__0232B434,
DoMoveConfuse, DoMovePoison, DoMoveToxic, DoMoveCurse, DoMoveWarp,
DoMoveLightScreen and DoMovePerishSong -- the same shape as the previous two
batches.
Three forward to helpers already declared in move_orb_effects.h (LowerSpeed and
TryInflictParalysisStatus), which fixes their argument meanings. The rest take
provisional externs written from the call site; TryWarp's third and fourth
parameters are left unnamed because nothing establishes what they are.
DoMoveParalyze__02328230 and DoMoveParalyze__0232B434 join
DoMoveParalyze__02326E80 from an earlier batch: three separate wrappers around
TryInflictParalysisStatus with identical arguments.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate. Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
None carries a region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveEndure, DoMoveUproar, DoMoveMist, DoMoveSafeguard, DoMoveTaunt,
DoMoveConversion2, DoMoveBlock, DoMoveWish, DoMoveIngrain and DoMoveSetDamage --
the same shape as the previous batch: forward to one status helper and return
TRUE.
All ten helpers are undeclared in the tree, so each takes a provisional extern
written from its call site. DoMoveBlock forwards to TryInflictShadowHoldStatus,
the same helper DoMoveShadowHold uses, with the same argument.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate. Confirmed by a matching build: build/pmdsky.us/pmdsky.us.nds: OK.
None carries a region conditional, so US alone is sufficient.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
DoMoveVitalThrow, DoMoveHealStatus, DoMoveEncore, DoMoveStringShot,
DoMoveFocusEnergy, DoMoveCounter, DoMoveParalyze__02326E80, DoMoveShadowHold,
DoMoveHaze and DoMoveBoostSpeed1 -- each a thin wrapper that forwards to one
status or stat helper and returns TRUE, following DoMoveDefendOrder's shape.
Three helpers were already declared in move_orb_effects.h, which fixes the
meaning of their constant arguments; the other seven take provisional externs
written from the call site.
Authored by Claude (Opus 5) under human direction. All ten matched on the first
candidate against a context preprocessed from the real headers. Confirmed by a
matching build: build/pmdsky.us/pmdsky.us.nds: OK. None of the ten carries a
region conditional, so US alone is sufficient here.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Split asm/overlay_29_022F05B4.s at 0x022F2B3C; add src/overlay_29_022F0EDC.c
and its header, and both objects to main.lsf.
Type corrections the match required:
- dungeon.field_0x1d8 / field_0x1dc: paired u16 fields -> struct position
- dungeon.field_0x614: u32 -> s32, tested >= 0
- display_data.leader_target_direction_mirror: enum direction_id -> u8; the
enum type folds the stored 0xFF to -1 and drops retail's mov r0, #0xff
- CanMonsterMoveInDirection's direction parameter: u16 -> s32; a u16 parameter
narrows the argument at the new call site, where retail passes it unchanged
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Not verified: the JP target, which is expected not to match -- the replaced asm
carries JAPAN-specific variants beyond the message-id offset. Two volatile
stand-ins remain in the new source, and this translation unit declares
DUNGEON_PTR as a scalar where other files declare it as an array; the two are
not byte-interchangeable here.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
IsMarowakTrainingMaze ov29_022EAEFC
FixedRoomIsSubstituteRoom ov29_022EAF20
IsSecretRoom SetUnkMusicFlag
IsDungeonEndReasonFailure ov29_0234969C
ChangeDungeonMusic ov29_023496B0
All ten reach struct dungeon through DUNGEON_PTR and every offset lands on a
named field: id at 0x748, hidden_outlaw_defeated_message at 0x2, unk_music_flag
at 0x792, dungeon_music_playing_id and field_0x2cb00 at 0x2CB06 and 0x2CB00,
gen_info.fixed_room_id at 0x40DA, and fainted_monster_dungeon_end_reason at
0x2CA66. No new types.
The constants resolve to existing enumerators that corroborate the function
names rather than being chosen to fit them: FIXED_SUBSTITUTE_ROOM is 110 and
FIXED_SECRET_ROOM is 113, which is what FixedRoomIsSubstituteRoom and
IsSecretRoom compare against; DAMAGE_SOURCE_ESCAPE is 633, the 0x279 in
IsDungeonEndReasonFailure. IsMarowakTrainingMaze tests the range
DUNGEON_NORMAL_FLY_MAZE to DUNGEON_EXPLORER_MAZE inclusive, 180 to 190.
IsDungeonEndReasonFailure reads its union member through an explicit (s16) cast.
enum damage_source_non_move declares no negative enumerator, so under -enum min
it is unsigned and compiles to ldrh with an unsigned comparison, where the target
has ldrsh and a signed one. The cast is local and reversible; adding a sentinel
to a shared enum, as was done for monster_id earlier on this branch, would not
be.
Two of the ten merged into src/run_dungeon_1.c, which already declares
DUNGEON_PTR as an array and reaches it as DUNGEON_PTR[0]. They follow that file's
convention rather than introducing a second spelling; both forms were checked on
a scratch and produce identical bytes.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
GetForcedLossReason GetHiddenStairsField
SetForcedLossReason SetHiddenStairsField
AreLateGameTrapsEnabledWrapper GetHiddenFloorField
GetSuccessfulExitTracker SetHiddenFloorField
SetDungeonEscapeFields GetMinimapData
All ten read or write struct dungeon through DUNGEON_PTR, and every offset lands
on a field dungeon.h already names: forced_loss_reason at 0x14,
successful_exit_tracker at 0x18, end_floor_no_death_check_flag at 0x8,
minimap_display_data at 0x1A264, and three fields of gen_info at 0x40C4 --
hidden_stairs_type, hidden_floor_type and fixed_room_id. No new types.
These are verified against the preprocessed dungeon.h context built in the
previous commit rather than a hand-written struct, which is why nine of the ten
matched on the first candidate instead of appearing to and then failing the
build. The setup cost of that context is now paid, so any overlay_29 function
reaching DUNGEON_PTR is cheap to verify; a scan finds 59 more small ones.
GetMinimapData needed the null check written as a conditional expression rather
than an if with an early return. The target predicates the whole thing --
addne twice for the non-null path, then moveq for the null one -- where the
statement form branches. That is the same predicate-ordering effect recorded on
the LFO handlers in 909484fb, in the direction that wants the expression form.
The US and JP offsets differ for the gen_info and minimap fields, which the asm
carries as #ifdef JAPAN. Nothing version-specific is needed in the C: the fields
are named the same in both, so dungeon.h resolves each build to its own offset.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
IsOutlawMonsterHouseFloor IsDestinationFloorWithMonster
IsGoldenChamber MissionTargetEnemyIsDefeated
IsJirachiChallengeFloor SetMissionTargetEnemyDefeated
IsDestinationFloorWithFixedRoom
IsDestinationFloorWithHiddenOutlaw
GetMissionEnemyMinionGroup
These are the functions the two previous commits could not land. The obstacle was
the scratch context, not the C, and this commit is the result of replacing it.
The context is now the real headers rather than a hand-written struct: mwccarm
-E over a file including global.pch, dungeon.h and mission.h, with
LM_LICENSE_FILE set, gives 14500 lines reproducing what the build actually sees.
Two things that took finding. global.pch is required -- without it util.h fails
at typedef s32 fx32_8, because s32 comes from nitro.h, which the build supplies
through -prefix rather than through any include in dungeon.h. And -E consumes
every macro, so TRUE, FALSE and NULL have to be restored by hand afterwards;
their values are taken from lib/include/nitro/types.h, NULL being ((void *)0) on
the C branch.
Against that context the earlier verdicts change. All eight functions landed in
the two previous commits still score 0, which is the check that the new oracle
agrees with the build. Eight of the thirteen unlanded ones score 0 unchanged and
are landed here as written. GetMissionEnemyMinionGroup does not: the real headers
let the compiler fold the +1 of enemy_species[index + 1] into the load
displacement, one instruction shorter than the target. Binding index + 1 to a
local first blocks the fold and scores 0, which is what is landed.
Four remain unmatched and are genuinely unmatched rather than mis-measured:
IsCurrentMissionType and IsCurrentMissionTypeExact at 515, where the target
branches to separate return blocks and every spelling tried so far produces
predicated moves; GetItemToRetrieve at 665 and GetItemToDeliver at 1530. Their
scores are identical under both contexts, so the context was never their problem.
The mission_type arguments use the enumerators from mission.h -- ARREST_OUTLAW,
EXPLORE_WITH_CLIENT, TAKE_ITEM_FROM_OUTLAW and CHALLENGE_REQUEST -- because the
build rejects the implicit conversion from int. The values agree with the
function names rather than being chosen to fit them.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
ov29_02349688 0x02349688
SetTargetMonsterNotFoundFlag 0x023496C4
GetTargetMonsterNotFoundFlag 0x023496D8
GetMissionTargetEnemy 0x02349620
The first three read and write the two dungeon flags at 0x1 and 0x3;
GetMissionTargetEnemy returns enemy_species[0]. No new types.
The previous commit left thirteen functions unlanded because they failed four
checksums with no compile error, and recorded the elimination so far. Bisecting
found a one-function reproducer, GetMissionEnemyMinionGroup, and disassembling
the object the real build emits identifies the cause.
The scratch and the real build disagree, and the scratch was wrong. For
enemy_species[index + 1] the target computes the index first --
add r0, r0, #1, then add r0, r1, r0, lsl #1, then add #0x700 and ldrsh [r0,
#0x6e]. The real build folds the constant into the displacement instead, giving
add r0, r1, r0, lsl #1, add #0x700, ldrsh [r0, #0x70], one instruction shorter
and the same address. Same compiler, same flags, same C: the difference is the
context. The scratch used a cut-down struct dungeon carrying only the offsets
these functions touch, and against that struct the compiler emits the target's
form; against the real header it folds.
That invalidates the scratch verification for the whole cluster, not just this
function. Score 0 against an invented struct is not evidence about a build that
uses the real one, even when every offset and size matches -- and they do, which
was confirmed separately by compile-time assertions against the real headers.
So the thirteen are not near-matches to iterate on; they need re-verifying
against a faithful context before any C is judged. A preprocessed context is
obtainable -- mwccarm -E on a file including dungeon.h and mission.h, with
LM_LICENSE_FILE set, yields 9200 lines that reproduce the real build's view --
but that output does not yet compile as scratch context on its own, failing at a
typedef, so wiring it up is unfinished.
The four here were each verified by a matching build rather than by the scratch:
three of them alone, GetMissionTargetEnemy alone, and all four together.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
ov29_02349188 0x02349188
GetMissionDestination 0x023491A4
ov29_023491B8 0x023491B8
IsDestinationFloor 0x02349208
All four reach DUNGEON_PTR->mission_destination, whose type is already fully
described in dungeon_mode.h. No new types.
This is the surviving part of a larger batch. Seventeen functions from this
cluster reached score 0 on a scratch; these four build matching in the tree and
the other thirteen do not, failing four checksums -- main, OVY_10, OVY_29 and
OVY_31 -- with no compile error. The thirteen are not in this commit.
What is known about that failure, so the next attempt does not repeat the work.
The struct layout is not the cause: compile-time assertions against the real
headers confirm mission_destination at 0x760, the two dungeon flags at 0x1 and
0x3, target_enemy_is_defeated at 0x1B, fixed_room_id at 0x16, enemy_species at
0xE, and sizeof(enum mission_type) == 1 and sizeof(enum monster_id) == 2 -- all
the offsets the scratch context assumed. A build of HEAD with no changes is
clean, so the regression is in the batch. Landing these four alone is clean.
Adding nine more, none of which use a mission_type enumerator, reproduces the
failure, which rules out the enumerator conversions that were the initial
suspicion. No pre-existing declaration of any of the seventeen names exists
elsewhere in the tree.
That main.sbin fails alongside the overlay points at something structural rather
than at a function body -- most likely the overlay size or the symbol exports in
the regenerated .inc files, since extract_function.py rewrites the .public list
when it splits an overlay .s. The byte-level diff of OVY_29 against a clean build
is the next step, using the cmp -l technique that located the rodata problem in
0f2e0ea8.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm, all reaching UNK_STORAGE_STRUCT_0X14:
sub_02043148 sub_0204329C sub_020433E0 ClearStorageSelectedItemTable
sub_0204316C sub_02043380 sub_02043400 CountSelectedStorageItems
sub_02043218 sub_02043398 sub_02043434 GetFirstSelectedStorageItemIndex
sub_0204322C sub_020433C0 sub_02043468
sub_0204323C sub_02043254
struct unk_020AFEE0 was introduced two dozen commits ago holding only the pointer
at 0x8 and filler before it. Its first eight bytes are now split into what these
functions read: halfwords at 0x0 and 0x2 and a pointer at 0x4, with a word at
0x10 added past the existing end. Nothing that already used field_0x8 moves.
The object behind that pointer is new, and its size is derived rather than
guessed. Its s16 array at 0x4 is bounded by 0x3e8 in three separate loops, which
is 1000 entries and ends at 0x7D3; the signed byte the collection-menu wrappers
read sits at 0x7D4, immediately after it. Two further fields at 0x910 and 0x18BC
come from sub_020430F4. Filling the gaps brings the struct to exactly 0x18C0,
which is the size in upstream's own name for its allocator,
InitUnkStorageStruct0x18c0. That agreement is the reason the layout is worth
trusting; the field names remain placeholders because nothing here establishes
meaning.
Fourteen of the seventeen matched on the first candidate. The other three are
loops whose index and pointer landed in swapped registers, with every
instruction otherwise correct, and all three were closed by reordering the local
declarations -- the same effect recorded in the previous two commits. All
orderings were enumerated rather than guessed: two locals for sub_02043254 and
GetFirstSelectedStorageItemIndex, three for CountSelectedStorageItems.
Eight callees remain asm and their prototypes are provisional, declared in the
shared header rather than repeated per file. None of them was declared anywhere
else in the tree. Two are typed from evidence rather than convention:
IsCollectionMenuActive and IsCollectionMenuState3 return bool8 because both
callers mask with and r0, r0, #0xff.
sub_020430F4 was attempted and is not included. It reaches the same struct and
its two extra fields are what sized it, but it stands at 495 after correcting
sub_0202C654 to the four arguments the target passes -- the fourth is a zero in
r3 that the first reading missed. Left in asm rather than landed as a near-match.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
SetPokemonBattled 0x0204FE58
GetNbItemAcquired 0x020503CC
SetPokemonBattled is the battled counterpart of SetPokemonJoined, already
decompiled in src/main_0204FDFC.c, and merges into that file beside it. It
differs only in which completion flag it sets and which flag array it indexes,
so the existing function is the template rather than anything derived here.
GetNbItemAcquired counts the set bits of items_acquired_flags across 0x580
entries. The pointer is hoisted to a local because the target loads it once
before the loop, and the word and bit indices use signed division and remainder,
which is what produces the asr/lsr/ror sequences the target has.
Both matched on the first candidate.
Seven adventure-log functions remain in asm. Four are large --
ComputeSpecialCounters at 144 instructions, CopyLogTo at 122, CopyLogFrom at 120
and ClearAdventureLogStruct at 74. The other three are not large but index the
region at 0x260 as an array spanning special_challenge_flags and the five
sentry_duty_game_points that follow it, mixing signed and unsigned division of
the same index within one function; expressing that without either misdescribing
the struct or writing something the compiler will not reproduce needs more care
than this batch had left.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm, all of them accessors on ADVENTURE_LOG_PTR:
SetAdventureLogStructLocation SetAdventureLogCompleted IncrementNbEvolutions
SetAdventureLogDungeonFloor GetAdventureLogCompleted GetNbEvolutions
GetAdventureLogDungeonFloor IsAdventureLogNotEmpty IncrementNbEggsHatched
IncrementNbDungeonsCleared GetNbDungeonsCleared GetNbEggsHatched
IncrementNbFriendRescues GetNbFriendRescues GetNbPokemonJoined
GetNbMovesLearned SetVictoriesOnOneFloor GetVictoriesOnOneFloor
GetNbPokemonBattled IncrementNbBigTreasureWins SetNbBigTreasureWins
GetNbBigTreasureWins SetNbRecycled GetNbRecycled
IncrementNbSkyGiftsSent SetNbSkyGiftsSent GetNbSkyGiftsSent
IncrementNbFainted GetNbFainted GetSentryDutyGamePoints
No new types. struct adventure_log already exists in include/adventure_log.h with
every field these touch named and offset-commented, and struct dungeon_floor_pair
in dungeon.h; every offset in the asm lands on a named field. That header is the
reason a batch this size was tractable: the only modelling needed was reading
which field each function touches.
Two orderings decided four of the thirty, both instances of the rule added to
MATCHING_TIPS in the previous commit. IsAdventureLogNotEmpty needed its loop
index declared before the pointer local, not after -- with the pointer first the
two land in swapped registers, at 35. The three Set functions needed the
completion-flag write placed before the clamp rather than after: written in the
obvious order the clamp is emitted ahead of the first pointer load and the two
literal-pool words come out reversed, at 250 each. Reordering the statement fixed
all three.
The counters clamp at 0x000F423F, decimal 999999, and each function sets one bit
of completion_flags[0]. The exact shape differs per function and is reproduced
rather than normalised: three increment then clamp, two clamp then increment, and
the three setters write the flag before storing the value. Those differences are
what the target does.
The clamp comparisons are signed even though the fields are u32, so each is
written with an explicit (s32) cast rather than retyping a shared struct.
ADVENTURE_LOG_PTR and _022AB69C move into adventure_log.h. src/main_0204FDFC.c
declared the pointer itself, from before any of this was decompiled; that line is
replaced by the header it already includes, and the object rebuilds unchanged.
Nine adventure-log functions remain in asm and are not in this commit:
ClearAdventureLogStruct, ComputeSpecialCounters, CopyLogTo, CopyLogFrom,
SetItemAcquired, GetNbItemAcquired, SetChallengeLetterCleared, SetPokemonBattled
and SetSentryDutyGamePoints.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
sub_02027A08 0x02027A08 sub_0202811C 0x0202811C
sub_02028080 0x02028080 sub_0202812C 0x0202812C
sub_020280C0 0x020280C0 DeleteWindow 0x02028194
sub_0202810C 0x0202810C sub_0202822C 0x0202822C
DeleteWindow explains _022A7A74, which the previous commit landed two accessors
for without knowing what it held. On deleting a window it walks the other
nineteen, and for every active one on the same background takes
base_tile + width * height, keeping the largest and a floor of 1, then stores
that at _022A7A74[bg_id]. So the pair is a next-free-tile watermark per
background, which also explains NewWindowScreenCheck setting the entry to 1 when
a background has no windows left: that resets the allocator.
Four of the eight return their callee's result rather than void, and only one of
them shows it. sub_020280C0 never touches r0 after calling sub_02027E30, which
is only consistent with r0 staying live to the return; typed void it scored 90,
typed s32 it scores 5, and the 5 is the literal-pool naming described below.
sub_0202810C and sub_0202811C are tail calls that score 0 either way, so their
own bytes settle nothing; they are typed s32 to match, and a reviewer should
read those two as following the family rather than as read off the target.
DeleteWindow needed two things past the obvious form. The background comparison
is bg == p->template.bg_id, not the reverse, which is worth 5 on its own. And
its four locals have to be declared i, top, p, bg: every one of the 24 orders
was tried, they range from 0 to 140, and only that one reaches 0. The
instructions were already identical at 35 -- the entire remaining difference was
which register held bg and which held i.
sub_0202812C and sub_02027A08 land at 10, and sub_020280C0 at 5, all of it the
literal-pool symbol naming: the target writes _022A8990, _022A8992 and _022A88E4
where C indexing WINDOW_LIST emits WINDOW_LIST+0xb4, +0xb6 and +0x8. Same
address, identical bytes once linked, at a flat 5 per word. The check for this
is in MATCHING_TIPS; the build is what settles it.
struct unk_020AFD4C is introduced for the 12-byte object at that address, sized
by the gap to _020AFD58. Only its word at 0x8 is touched here, a bitmask that
five of these functions set a bit in, indexed by bg_id.
DeleteWindow had a provisional declaration in include/main_0202AAA8.h from the
menu commits, with three decompiled callers. That is replaced by an include of
the new header and all three objects rebuild.
sub_0202836C now has a fifth declaration in the tree, and the existing four do
not agree: overlay_15 says int, overlay_24_end says s32, and overlay_24_init and
overlay_25_init say s8. It is declared s32 here, in the caller's own header
rather than in window.h, so that no overlay including window.h sees a conflicting
one. They should collapse into a single header when the callee lands.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build with the three DeleteWindow callers rebuilt:
build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
sub_02027624 0x02027624 ClearWindow 0x02027B58
UpdateWindow 0x02027AF0 sub_0202825C 0x0202825C
sub_02027B1C 0x02027B1C sub_02028270 0x02028270
sub_020282C8 0x020282C8 sub_020282F4 0x020282F4
sub_0202830C 0x0202830C sub_02028324 0x02028324
Four bss symbols these functions index at stride 0xE0 are aliases into
WINDOW_LIST[0], not separate objects: _022A88E4 is +0x8, _022A88F0 is +0x14,
_022A88F8 is +0x1C and _022A8994 is +0xB8. The block from WINDOW_LIST to the end
of _022A8994 measures 0xB8 + 0x10C8 = 0x1180 = 20 * 0xE0, and every one of those
boundaries falls on a field boundary of the Window layout added earlier. That is
independent corroboration of the layout, from the linker rather than from the
struct definition.
Five of the ten do not reach score 0 on a scratch, and are landed anyway. Their
instructions all match; the only difference is the literal-pool word, which the
target names _022A8994 where C indexing WINDOW_LIST emits WINDOW_LIST+0xb8. Same
address, same relocation target, identical bytes once linked -- the scratch diff
compares symbol names, at a flat 5 per word, so ClearWindow and sub_02027B1C
report 10 and the three single-literal ones report 5. The matching build is what
settles it.
Which literal appears is decided by whether the index is constant. A constant
offset keeps the base symbol in the pool and puts the offset in the instruction;
a variable index folds the offset into the pool word because the index has to be
scaled separately. That rule cost a build here: NewWindowScreenCheck stores to
_022A7A6C at #8 and #0xa, and rewriting those two lines as _022A7A74[0] and [1]
to remove an apparent duplicate changed the pool word and failed main.sbin, with
no compile error and no overlay cascade. It is reverted, and both declarations
are kept, because the target keeps both.
sub_020282C8 writes width*8 and height*8 through an out parameter. It reuses
Point rather than adding a second two-s32 struct; the layout is identical and
the type already exists, but a size is not a coordinate, so read that as
structural reuse and not as a claim about meaning.
Two prototypes in the tree contradicted the ones landing here and are replaced
with includes: overlay_31_02382820.c declared UpdateWindow itself, and
overlay_31_02383880.c declared sub_020282F4 taking s8. Both objects are rebuilt
to confirm the retyping is byte-neutral rather than assumed.
Two more are left alone and are worth a later pass. overlay_13_0238BDA8.c
declares UpdateWindow and sub_02027B1C taking s8, and overlay_25_init.c declares
both taking char *; the second is a genuine mistyping, since the value it passes
is a window id, but correcting it means retyping ov25_0238B414's own parameter
and its callers, which is a larger change than this batch. The s8 declaration in
overlay_13 carries an upstream annotation identifying sub_02027B1C, which is
worth keeping in place rather than deleting to make a byte-neutral edit.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build with both overlay_31 objects rebuilt: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
NewWindowScreenCheck 0x02027648
GetPaletteBaseAddress__020278A8 0x020278A8
SetScreenWindowsColor 0x02027A80
NewWindowScreenCheck counts the active windows on each background before
delegating to NewWindow, and sets a flag when either screen has none. It reads
is_active as a signed byte at 0xB6 and bg_id at 0x8, both named by the Window
layout added in the previous commit, so the loop reads as what it does.
It walks with a cursor rather than indexing. Indexing WINDOW_LIST[i] emits an
mla per iteration; the target advances a pointer by 0xE0 at the bottom of the
loop. Its two counters also have to be assigned in the order sub then main --
declaration order alone was not enough, and every permutation of the four locals
left between 20 and 95.
SetScreenWindowsColor's colour parameter is s32, not u8, and that distinction is
not visible in its own bytes: both score 0 against it, because the store is a
strb either way. It is visible one level up. SetBothScreensWindowsColor, landed
matching in the previous commit, passes its argument straight through; a u8
parameter makes the compiler truncate at that call site, which changes an object
that already matched, changes the size of main.sbin, and shifts every overlay
after it. The first build of this batch failed 33 checksums for that reason,
with no compile error and nothing wrong in the function being added.
GetPaletteBaseAddress__020278A8 reads through _020AFC70, which include/
main_02064FFC.h already declares as u8 *. It is declared the same way here
rather than being given a second, differently typed declaration.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
SetBothScreensWindowsColor 0x02027A80
GetWindowRectangle 0x02028284
GetWindowContents 0x0202833C
GetWindowContents is the function the menu accessors landed over the last several
commits all call. It reads the pointer at 0xC of a WINDOW_LIST entry, and its
callers cast that to whichever menu layout they use.
struct Window gains the fields these two touch: bytes at 0x4, 0x5, 0x7 and 0x8
around the existing width at 0x6, and the pointer at 0xC. The trailing padding
shrinks to keep the entry at 0xE0, which is the stride GetWindow indexes by.
GetWindowRectangle converts four of those bytes into a rectangle in pixels,
multiplying each by 8, writing top and bottom from 0x5 and 0x7 and left and
right from 0x4 and width. Its out parameter has no existing type and becomes
struct unk_02028284.
GetWindowContents needed the entry address bound to a pointer local. Returning
WINDOW_LIST[window_id].field_0xC folds the field offset into the literal pool
entry, giving a .word WINDOW_LIST+0xc and an indexed load; the target keeps
WINDOW_LIST as the literal and reaches the field with mla plus ldr [r1, #0xc].
SetScreenWindowsColor is still asm and had no declaration in the tree, so its
prototype is provisional.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
GetWindow 0x020275F8
It indexes WINDOW_LIST with a stride of 0xE0, so its argument is a window id
rather than a pointer, and its result is the address of one entry.
overlay_31_02382820.c declared it as taking a struct Window * and passed one,
which cannot be what the target computes: multiplying a pointer by 0xE0 and
adding it to WINDOW_LIST is meaningless. The value it passes is an id, and the
same value goes to DrawTextInWindow and UpdateWindow, so those take ids too.
This retypes the three window callbacks and both of those prototypes
accordingly, along with the callback type CreateTextBox stores.
That retyping is byte-neutral -- a pointer and an id both travel in r0 -- but
overlay_31_02382820.o was already matching, so it is rebuilt here to confirm
rather than assumed.
struct Window keeps its existing return type and gains padding to its real 0xE0
size, which is what lets &WINDOW_LIST[id] stride correctly. Its first seven
bytes are unchanged, so the width field overlay_31 reads is where it was. The
struct moves from the overlay's header into include/window.h, which already
exists and is where a main-binary function returning one can reach it; the
overlay includes it rather than the main binary including an overlay header.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build with overlay_31_02382820.o rebuilt:
build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
AllocUnkBagStruct 0x02042B98
FreeUnkBagStruct 0x02042BBC
SetAreaNameBoxState3 0x0202FD3C
GetKeyboardStringResult 0x0203755C
AllocUnkBagStruct and FreeUnkBagStruct manage the pointer at offset 8 of
UNK_STORAGE_STRUCT_0XC, the field the selection accessors in the previous commit
read. The allocation is 0xC8 bytes, which is 0x32 words -- the same count
ClearBagSelectedItemTable zeroes and the same inventory size the bag walkers
use.
MemAlloc is still asm and has no owning header, so its prototype is provisional
and declared beside the caller, matching the signature overlay_31_02382820.c
already uses. That is now a second declaration of it; both should collapse into
one header when MemAlloc lands.
SetAreaNameBoxState3 writes a word at 0xA0 of the window contents, so the filler
before 0x198 in struct unk_0202AAA8 is split to expose it. Nothing after it
moves.
GetKeyboardStringResult reads through the unnamed pointer _020AFDF0, which keeps
its placeholder name, into a word at 0xF8 of whatever it points at.
GetWindow was looked at and deliberately left in asm. It indexes WINDOW_LIST
with a stride of 0xE0, so its argument is an index, but overlay_31_02382820.c
declares it as taking a struct Window * and passes one. Landing it correctly
means retyping that caller, and struct Window there describes only its first 7
bytes, so the two disagree about more than this function.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
ClearBagSelectedItemTable 0x02042AF8
IsBagItemIndexSelected 0x02042B84
IsStorageItemIndexSelected 0x02043568
GetNotifyNote 0x020484A0
SetNotifyNote 0x020484B0
EventFlagBackupVeneer 0x02048758
UNK_STORAGE_STRUCT_0XC and UNK_STORAGE_STRUCT_0X14 already carry those names in
asm. Only their field at offset 8 is touched here, a pointer to an array, so
they are typed with placeholder structs holding that field and filler before it.
The element widths come from the loads: words for the bag table, bytes for the
storage one.
Their struct names use the globals' own addresses, 0x020AFED4 and 0x020AFEE0,
derived by walking back from UNK_STORAGE_STRUCT_0X8_PTR_1, which sits at the
labelled address 0x020AFEF4, through the two definitions' byte counts. That is
arithmetic on the data rather than a lookup, and a reviewer may want to confirm
it; nothing in this commit depends on the names being right, only on the layout.
ClearBagSelectedItemTable zeroes 0x32 entries, the same count the bag walkers in
earlier commits use for an inventory.
EventFlagBackup is still asm and had no declaration in the tree, so its
prototype is provisional.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
SetCollectionMenuField0x1BC 0x0202C5E0
SetCollectionMenuField0x1C8 0x0202C794
SetCollectionMenuField0x1A0 0x0202C7A8
SetCollectionMenuField0x1A4 0x0202C7BC
SetCollectionMenuVoidFn 0x0202C7D0
SetCollectionMenuField0x1B2 0x0202D0D8
These reach the same buffer as the parent, simple and advanced menu accessors,
through the same GetWindowContents, but they do not share its layout: this set
writes a word at 0x1A0 where CheckParentMenuField0x1A0 reads a byte there, and a
byte at 0x1B2 which falls inside the word the advanced text box uses at 0x1B0.
So they are given their own view, struct unk_0202C5E0, named for the
lowest-addressed function that takes it, rather than forcing one struct to
describe both. GetWindowContents returns void *, so each menu type casting the
buffer to its own layout is consistent with what the target does; a single
merged struct would have to assert that the two layouts agree, which the stores
show they do not.
Two of the offsets are inferred from the store width alone and nothing else:
0x1A8 is written with a word and the function is named VoidFn, so it is typed
void *, but the target would look the same for any pointer or 32-bit value.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
SetSimpleMenuField0x1AC 0x0202BA0C
CloseAdvancedMenu 0x0202BC44
IsAdvancedMenuActive2 0x0202BCBC
SetAdvancedTextBoxField0x1C4 0x020307A4
SetAdvancedTextBoxField0x1C2 0x0203083C
SetAdvancedTextBoxState5 0x0203088C
struct unk_0202AAA8 gains fields at 0x1AC, 0x1BC, 0x1C2, 0x1C3 and 0x1C4. The
filler that stood between 0x1A8 and 0x1B0 is split to expose 0x1AC; nothing
after it moves.
CloseAdvancedMenu differs from the simple- and parent-menu versions: it frees
only the window contents, not the pointer at 0x198 those two free first. That is
what the target does and is not an omission here.
IsAdvancedMenuActive2 tests the same states as IsSimpleMenuActive, 7 and 8, and
like it binds the state to a local so the field is loaded once.
SetAdvancedTextBoxField0x1C2 stores a constant 1 rather than an argument, so it
takes only the window id.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
GetAdvancedMenuCurrentOption 0x0202BCFC
GetWindowIdSelectedMenuItemIdx 0x0202C748
IsDialogueBoxActive 0x0202F180
GetWindowIdPageStart 0x02030A18
GetAdvancedTextBoxFlags2 0x02030A40
GetSelectedMenuItemIdx 0x02032578
GetSelectedMenuItemIdx computes the absolute item index from the paging struct
added in the previous commit, current page times items per page plus the
selection within the page, which the target folds into one mla.
GetWindowIdPageStart, GetWindowIdSelectedMenuItemIdx and
GetAdvancedMenuCurrentOption resolve a window and forward to the paging
functions at offset 4 of its contents. The last two have identical bodies and
are decompiled as written rather than one calling the other, since that is what
the target does.
struct unk_0202AAA8 gains two fields the other two functions read: a byte at 0x8
and a word at 0x1B0. The filler before 0x198 is split to expose the first
without moving anything after it.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
GetSelectedItemOnPage 0x02032558
GetCurrentPage 0x02032560
GetPageStart 0x02032568
GetTotalNumMenuItems 0x0203258C
GetNumItemsOnPage 0x02032594
GetMaxItemsOnPage 0x0203259C
GetTotalNumPages 0x020325A4
Six read one word each from a struct with no existing type, introduced here as
struct unk_02032558 with placeholder fields at 0xBC through 0xD0. GetPageStart
multiplies two of them, current page by max items per page, which is consistent
with the names but is an observation about the arithmetic rather than something
the bytes label.
GetSelectedItemOnPage was declared provisionally in include/main_0202AAA8.h last
commit, taking void *, because it was still asm. That declaration is removed and
its caller now includes the real header. The argument reaching it there is a
void * offset by 4 from the window contents, which converts implicitly; the
caller's object is rebuilt here and still matches.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
GetWindowIdSelectedItemOnPage 0x0202AB80
CloseSimpleMenu 0x0202B4C4
IsSimpleMenuActive 0x0202B520
CheckSimpleMenuField0x1A0 0x0202B540
GetSimpleMenuField0x1A4 0x0202B558
These are the simple-menu counterparts of the parent-menu accessors in the
previous commit and reach the same window contents through GetWindowContents,
so they reuse struct unk_0202AAA8, which gains a field at 0x1A4.
IsSimpleMenuActive tests states 7 and 8 where the parent-menu version tests 8
and 9. As there, the state is bound to a local: comparing the field against two
values inline emits two loads and the target loads it once.
DeleteWindow moves into the shared header and MemFree is taken from
main_02001188.h, rather than each source declaring them again. Two of these
functions need both, and a second local copy of a prototype is the kind of
disagreement no build can report.
The asm files this split produces keep upstream's jump-table annotations. An
earlier run of precommit.py stripped them, having discarded the removals that
rule 2 matches against; that is fixed separately in the workspace repo, and the
158 comments in this split are intact here.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
IsEmptyString 0x0202A66C
SetParentMenuState7 0x0202AAA8
CloseParentMenu 0x0202AABC
IsParentMenuActive 0x0202AB40
CheckParentMenuField0x1A0 0x0202AB60
The four menu functions all resolve their window through GetWindowContents,
which src/overlay_10_022BCC60.c already declares as returning void *. What it
returns has fields at 0x198, 0x19C and 0x1A0 and no existing type, so it is
introduced as struct unk_0202AAA8, named for the lowest-addressed function that
uses it, with placeholder fields.
The struct and the GetWindowContents prototype live in include/main_0202AAA8.h
and are included by the second source rather than repeated in it. Two copies of
a struct definition in two translation units is the same drift risk as two
prototypes, and nothing in the build would report them disagreeing.
Return widths differ across these and are read off the target. IsEmptyString and
CheckParentMenuField0x1A0 end in and r0, r0, #0xff, so they return bool8;
IsParentMenuActive does not, so it is word-sized despite reading like a
predicate.
IsParentMenuActive also needed its state field bound to a local. Comparing
menu->field_0x19C against two values inline emits two loads; the target loads it
once.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
PlayBgmByIdVeneer 0x02017B58
PlayBgmByIdVolumeVeneer 0x02017B64
PlaySeByIdVolumeWrapper 0x02017C80
GroupOamAttributesWrapper 0x0201BA9C
CopyAttributesToOamWrapper 0x0201BAAC
CopyAndInterleaveWrapper 0x0201BFF0
SetAnimationControlPausedFlag 0x0201D198
DeleteWanTableEntryVeneer 0x0201D72C
Most are tail calls the compiler reaches with bx rather than bl, either passing
their arguments through unchanged or supplying one: PlaySeByIdVolumeWrapper
fixes the volume at 0x100, the two Oam wrappers offset their argument by 0x20,
and CopyAndInterleaveWrapper halves its length.
That halving is a shift, not a division. Written as len / 2 the compiler adds the
round-toward-zero correction a signed divide needs; the target has a bare asr #1,
which is what len >> 1 produces.
SetAnimationControlPausedFlag sets or clears bit 0x4000 of the bitfield at the
start of struct animation_control, which already exists in graphics.h.
Two cautions for a reviewer. The callees are all still asm and had no
declarations in the tree, so every prototype here is provisional and declared
beside its caller. And a veneer that only forwards constrains nothing about its
own argument list -- the arguments are already in the right registers, so
declaring none, one or several all produce the same bytes. The parameter lists
are chosen to read sensibly against each callee's name, not read off the target.
The exception is PlaySeByIdVolumeWrapper, whose signature is pinned by an
existing C caller in overlay_25_init.c that passes a single s32.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
SetMoneyStored 0x0201070C
GetCurrentKecleonShop1ItemByIndex 0x02010898
GetCurrentKecleonShop2ItemByIndex 0x02010BA4
GetExclusiveItemOffset 0x02010E40
SwapShopFreeDoublePointer 0x020114F8
IsMonsterAffectedByGravelyrockGroundMode 0x02011830
The two Kecleon accessors index arrays hanging off pointers at 0x132C and
0x1370. Unlike the previous two extensions those offsets fall inside
fill2[0x1009] rather than past the struct's end, so the filler is split into
three runs with the two pointers between them. maybeMoney still lands at 0x1394
and nothing after it moves, which the rebuild of every struct bag_items user
confirms.
SetMoneyStored clamps to 0x0098967F, decimal 9999999, and to zero below.
GetExclusiveItemOffset returns zero unless the item is in
CATEGORY_EXCLUSIVE_ITEMS, which item.h already defines as 15, and otherwise the
item's distance from 0x1bc, narrowed to 16 bits.
IsMonsterAffectedByGravelyrockGroundMode normalises the species through
FemaleToMaleForm and compares against two ids, left as raw values because naming
them is not this commit's job. Its return is word-sized: bool8 appends
and r0, r0, #0xff, which the target does not have.
SwapShopFreeDoublePointer frees through two levels of indirection and clears the
outer pointer, returning early when it is already null.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Decompile from asm:
AddMoneyStored 0x02010758
SetEggSpecies 0x02010794
SetUnkGameState0x13a6 0x020107C4
SetEggHatchTimer 0x020107F4
DecrementEggHatchTimer 0x0201080C
GetRecycleItemId 0x02011DF0
RecycleItemHasTradeTypePrizeTicket 0x02011DFC
GetRecycleItemBonusOdds 0x02011E18
ClearRecycleShopOffer 0x02011F14
GetGameStateRecycleCount 0x02011F30
GetRankForRecycleShop 0x02011F48
GetRecycleOfferCooldown 0x0201227C
Nine of these reach BAG_ITEMS_PTR_MIRROR. Five are the setters matching getters
already landed; the other four use offsets past them, so struct bag_items gains
four more fields, up to 0x13B4. As with the previous extension they are appended
past the end, and the field at 0x13AC is left to align naturally rather than
declaring a byte at 0x13AB that nothing reads.
AddMoneyStored tail-calls SetMoneyStored, which is still asm; its prototype is
provisional. DecrementEggHatchTimer only decrements a non-zero timer, which the
target expresses with predicated subne/strneh rather than a branch.
The remaining three take a pointer to a pointer to a small struct with fields at
0x0, 0x4 and 0x12. No existing type matches that layout, so it is introduced as
struct unk_02011DF0, named for the lowest-addressed function that takes it, with
placeholder fields. The bytes between 0x8 and 0x12 are unexamined and left as
filler.
Every one of the twelve matched on the first candidate.
No comments are added to any pmd-sky file.
Authored by Claude (Opus 5) under human direction. Confirmed by a matching
build: build/pmdsky.us/pmdsky.us.nds: OK.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
