-
Notifications
You must be signed in to change notification settings - Fork 38
Match 600+ P2 functions & VU0/splice foundations #263
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 131 commits
7fbc500
8798f9f
eb9e062
2fcad9e
19bb095
49923bd
e7f9555
6e3b233
310ddbf
0645253
b19aae2
de69664
ac73e43
221133e
6ddec1f
95f11db
92efbf5
b30a221
5bd1f87
09aad7c
3381a3b
cfbeaa6
25b46e6
0815234
dfbad75
7ad2fd6
e9d8103
ed8f0f0
3b845dd
d04bfc4
af95e87
687c52e
babfe1a
0daedd7
127633a
80d641c
922a1f1
b97e0fc
44cb5fc
19c1080
324ac18
75b21ab
171975d
8eaae8e
b6e743a
8b6ca96
e3b5a38
2c96a8f
7375164
4310482
f0f1515
0af2f42
312de9d
d4e46f0
6423975
4b7be8a
37f1595
d5b696f
b5a6f17
8e07332
e595210
03b1cf8
c3ce690
2e1050e
08c1922
dd41e0a
a9e4cb0
78ca57d
c030d7d
0da2988
e2506db
6cc2cd7
d2b0ea9
8ce4de0
8d152d3
8de0bb8
eb36c4e
3e31d64
061aa14
42d8cb7
450d613
a2da977
bb530d7
19b46c0
4031d29
399a44b
51e84ac
a5564b9
6204e06
3515ac0
c0d13a7
4072809
100044d
96b5842
b5ed65e
9b114c3
15aebb9
15da083
ac26d64
e8eea34
aca5b98
eecf79b
3761615
9e3679f
ba7aeb7
f90a8a5
b22a845
b87198d
e2a4d4f
7b7bd77
e6dca32
159316c
257e1ab
c436f8a
29ed5ae
57f0b8c
4ddec33
e02bf9b
459b0b1
5a38764
11862c5
1cdd9cc
95b770e
ab0cc5e
f636d05
3e67f3a
4eab3db
0a6cc84
8c68053
0db4c60
22dcd68
2b80af6
c4e8657
09b96a8
fcf131e
4804054
3297fe1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please re-add the deleted comments |
|
TheOnlyZac marked this conversation as resolved.
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -256,6 +256,11 @@ void StartSound(SFXID sfxid, AMB **ppamb, ALO *palo, VECTOR *ppos, float sStart, | |
| */ | ||
| void StopSound(AMB *pamb, int msRampdown); | ||
|
|
||
| // The symbol's GCC2 mangling says (int, unsigned char), but the ROM call | ||
| // sites pass the second argument as a plain word with no narrowing, so the | ||
| // in-game declaration must have taken ints; bind by literal symbol name. | ||
| extern "C" void SetAMRegister__FiUc(int n, int bReg); | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not sure what to say except this should be C++ and the name unmangled regardless of what's happening at the callsites.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The symbol is likely taken from the prototype build and has since changed in the retail build. |
||
|
|
||
| /** | ||
| * @brief TODO | ||
| */ | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -43,6 +43,12 @@ extern CSplotheap g_splotheapUnk1; | |
| extern CSplotheap g_splotheapProc; | ||
| extern CSplotheap g_splotheapMethod; | ||
|
|
||
| // Sidebag binding-node helpers: allocate a node {int n; CRef ref; pNext} | ||
| // from g_splotheapUnk1, and recursively clone a node list. | ||
| class CFrame; | ||
| extern "C" void *FUN_0011C498(); | ||
| extern "C" void FUN_0011C418(void *psbbFrom, void *psbbTo, CFrame *pframe); | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why are these |
||
|
|
||
| static void *PvFromPsplot(SPLOT *psplot); | ||
| static SPLOT *PsplotFromPv(void *pv); | ||
| bool FIsPvGarbage(void *pv); | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -260,6 +260,8 @@ void SetSggSggs(SGG *psgg, SGGS sggs); | |
|
|
||
| void AssignSggSearchPoints(SGG *psgg); | ||
|
|
||
| struct PUFFC; | ||
| extern "C" void FUN_001c8920(PUFFC *ppuffc); | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No extern C |
||
| /** | ||
| * @brief Checks the struct has a 0 on an address, no idea what it means. | ||
| * @todo Figure out what this actually does and give it a better name. | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -228,30 +228,6 @@ JUNK_ADDIU(10); | |
| JUNK_ADDIU(10); | ||
|
|
||
| INCLUDE_ASM("asm/nonmatchings/P2/989snd", snd_GotReturns__Fv); | ||
| #ifdef SKIP_ASM | ||
| // Rodata | ||
| int snd_GotReturns(void) | ||
| { | ||
| FlushCache(0); | ||
| if (gCommBusy == NULL) { | ||
| return 1; | ||
| } | ||
|
|
||
| if (sceSifCheckStatRpc(&gSLClientData.rpcd)) { | ||
| return 0; | ||
| } | ||
|
|
||
| if (*gCommBusy == -1 && (gCommBusy[gAwaitingInts + 1] == -1)) { | ||
| gCommBusy = NULL; | ||
| return 1; | ||
| } else { | ||
| printf("989snd.c: Sif says RPC isn\'t busy, but we still don\'t have returns from the IOP!\n"); | ||
| return 0; | ||
| } | ||
|
|
||
| return 1; | ||
| } | ||
| #endif | ||
|
Comment on lines
-231
to
-254
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Unnecessary deletion
TheOnlyZac marked this conversation as resolved.
|
||
|
|
||
| void snd_PrepareReturnBuffer(u_int* buffer, int num_ints) | ||
| { | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are these and why are they extern C?