Skip to content

OK 7B87B0 - #60

Open
Drahsid wants to merge 2 commits into
masterfrom
OK_overlay3_7B87B0
Open

OK 7B87B0#60
Drahsid wants to merge 2 commits into
masterfrom
OK_overlay3_7B87B0

Conversation

@Drahsid

@Drahsid Drahsid commented Jul 29, 2026

Copy link
Copy Markdown
Owner

another overlay 3 file

@queueRAM queueRAM left a comment

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.

Looks good. Just had a couple suggestions, assuming it still matches.

Comment thread src/overlay_3/overlay_7B8760/7B87B0.c Outdated
Comment thread src/overlay_3/overlay_7B8760/7B87B0.c Outdated
Comment thread src/overlay_3/overlay_7B8760/7B87B0.c Outdated
Drahsid and others added 2 commits July 28, 2026 21:35
Co-Authored-By: queueRAM <129774+queueRAM@users.noreply.github.com>
@Drahsid
Drahsid force-pushed the OK_overlay3_7B87B0 branch from df30c9c to 520366d Compare July 29, 2026 01:40
Comment thread include/actor.h

// actor 0 is always the main character, Marina
#define gPlayerActor gActors[PLAYER_INDEX]
#define gPPlayerActor (gActors)

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.

What's the intended purpose of the addition of gPPlayerActor?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

line 544 7B87B0.c requires gActors-> to match.

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.

Got it, so gPPlayerActor is just providing a more meaningful name than gActors in those instances

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