Skip to content

Decomp overlay44 02235340 and 02232E9C#464

Closed
ferwardle wants to merge 30 commits intopret:masterfrom
ferwardle:decomp/overlay_44_02232E9C
Closed

Decomp overlay44 02235340 and 02232E9C#464
ferwardle wants to merge 30 commits intopret:masterfrom
ferwardle:decomp/overlay_44_02232E9C

Conversation

@ferwardle
Copy link
Copy Markdown
Contributor

No description provided.

Comment on lines +199 to +201
if (arg0->unk6CC == NULL) {
GF_AssertFail();
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All if { GF_AssertFail } might be replaceable with GF_ASSERT() directly.
Example here: GF_ASSERT(arg0->unk6CC != NULL);


FS_EXTERN_OVERLAY(OVY_42);

void ov44_02232EA8(OverlayManager* overlayMananger) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Surely is ov44_Init (name suggestion)

temp_r0->unk27 = 1;
}

s32 ov44_02232F64(OverlayManager* overlayManager, u32* arg1) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Then ov44_Main (the main loop, with the state machine)

return 0;
}

s32 ov44_022330A8(OverlayManager* arg0) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Finally ov44_Exit (when overlay is unloaded, freeing all allocated objects etc.)

@adrienntindall
Copy link
Copy Markdown
Collaborator

This has merge conflicts with your other PR that got submitted. Usually to prevent this going forward you'll want to have one PR be a branch of the other.

@red031000
Copy link
Copy Markdown
Member

please resolve conflicts, also, nathan's suggestions are a good point for improvement

@ferwardle
Copy link
Copy Markdown
Contributor Author

Going to close this for now until I finish the rest of overlay_44 to make sure I get the correct file boundaries

@ferwardle ferwardle closed this Apr 14, 2026
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.

4 participants