Skip to content

Match/app stuff#75

Merged
gcsmith merged 12 commits intogcsmith:mainfrom
Bl00D4NGEL:match/app-stuff
Mar 3, 2026
Merged

Match/app stuff#75
gcsmith merged 12 commits intogcsmith:mainfrom
Bl00D4NGEL:match/app-stuff

Conversation

@Bl00D4NGEL
Copy link
Contributor

  • matches all of app/code_CB7C0.c

  • matches all of app/code_CD9D0.c

  • rename app/code_CB7C0.c to app/targets.c

  • rename app/level_objects.c to app/task.c

Copy link
Contributor

@queueRAM queueRAM left a comment

Choose a reason for hiding this comment

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

Looks pretty good, just a couple minor comments

#include <uv_matrix.h>
#include <uv_sched.h>

#define WORLD_MODEL_ID 0xFFFF // used to indicate a model that's not bound to a level?
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if related, but code in A6000.c considers model id 0xFFFF invalid for its path purposes. I think your name choice makes sense, but I just wonder what it means there.

if (arg1->modelId == 0xFFFF) {
    _uvDebugPrintf("uvPathPoseLine: Path has an invalid Model ID!\n");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I suppose something related to "world models" not being.. pathable? or something

@gcsmith gcsmith merged commit 94b25db into gcsmith:main Mar 3, 2026
1 check passed
Bl00D4NGEL added a commit to Bl00D4NGEL/Pilotwings64Decomp that referenced this pull request Mar 3, 2026
* match: func_803464A0

* rename app/level_objects to app/task, rename app/code_CB7C0 to app/target

* match the rest of targets.c

* match: app/code_C9B60 func_80342630..func_80343294

* match: rest of app/code_C9B60

* misc splat fixes

* misc changes

* fix build

* misc documentation

* refac: replace Unk8037DCA0_Unk4C with Unk8037F098
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.

3 participants