Skip to content

gActorSetup{Opa,Xlu}DL docs - #1898

Merged
AngheloAlf merged 1 commit into
zeldaret:mainfrom
Dragorn421:actorsetupdldocs
Aug 30, 2026
Merged

gActorSetup{Opa,Xlu}DL docs#1898
AngheloAlf merged 1 commit into
zeldaret:mainfrom
Dragorn421:actorsetupdldocs

Conversation

@Dragorn421

Copy link
Copy Markdown
Contributor

fuller explanation here https://discord.com/channels/688807550715560050/873211000122384415/1538758944376492083
I left a lot of details out because I don't think random comments in the codebase should be n64 graphics tutorials unfortunately

also while researching I noticed one odd usage of gActorSetupOpaDL, the octoroks EnOkuta use it as a "empty dlist" rather than with opa semantics which suggests gActorSetupOpaDL may not have been the original name, but w/e

@hensldm hensldm added documentation Improvements or additions to documentation Needs-second-approval Second approval labels Aug 29, 2026

@AngheloAlf AngheloAlf left a comment

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.

also while researching I noticed one odd usage of gActorSetupOpaDL, the octoroks EnOkuta use it as a "empty dlist" rather than with opa semantics which suggests gActorSetupOpaDL may not have been the original name, but w/e

Or maybe the dev just noticed they already had an empty dl and decided to reuse it even if it wasn't intended as one. We'll never know

@hensldm hensldm added Merge-ready All reviewers satisfied, just waiting for CI and removed Needs-second-approval Second approval labels Aug 30, 2026
@AngheloAlf
AngheloAlf merged commit ff5b972 into zeldaret:main Aug 30, 2026
5 checks passed
@Dragorn421
Dragorn421 deleted the actorsetupdldocs branch August 30, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Merge-ready All reviewers satisfied, just waiting for CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants