Skip to content

Decompile overlay 44#478

Merged
adrienntindall merged 27 commits into
pret:masterfrom
ferwardle:decomp/overlay_44_02232E9C
May 18, 2026
Merged

Decompile overlay 44#478
adrienntindall merged 27 commits into
pret:masterfrom
ferwardle:decomp/overlay_44_02232E9C

Conversation

@ferwardle
Copy link
Copy Markdown
Contributor

Wasn't able to get most of the data to match because of ordering, so left them in overlay_44.s but included comments in the C files with the decompiled data.

Copy link
Copy Markdown
Collaborator

@adrienntindall adrienntindall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly nitpicks, don't forget to also run format.sh

Comment thread include/overlay_00_thumb.h Outdated
Comment thread include/overlay_42.h Outdated
Comment thread src/overlay_44_0222CDAC.c Outdated
Comment thread src/overlay_44_0222CDAC.c Outdated
Comment thread src/overlay_44_0222CDAC.c Outdated
Comment thread src/overlay_44_0222CDAC.c Outdated
Comment thread src/overlay_44_0222CDAC.c
}

s32 ov44_0222EA2C(UnkStruct_ov44_022319EC* arg0, s32 arg1) {
ListMenuTemplate spC; // Unknown struct of size 0x20
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can remove comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I used ListMenuTemplate as a placeholder type that fit the size, still unknown what type sub_0203A1C4 actually expects

Comment thread src/overlay_44_0222CDAC.c Outdated
Comment thread src/overlay_44_0222CDAC.c Outdated
GfGfx_EngineBTogglePlanes(8, 1);
}

void ov44_02230300(UnkStruct_ov44_022319EC* arg0, enum HeapID heapID) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's clear that this and the below functions print records messages, but I'm not sure off the top of my head where this would be in the game. Could be a good clue to start documenting the file as a whole though. Maybe in some multiplayer setting?

Comment thread src/overlay_44_02232E9C.c Outdated
Copy link
Copy Markdown
Collaborator

@adrienntindall adrienntindall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's still some documentation work I see that can be done but this PR is big enough already that it can be handled at a later date when someone wants to go through and document everything properly

@adrienntindall adrienntindall merged commit 0b2dab8 into pret:master May 18, 2026
1 check passed
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.

2 participants