-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Remaining functions to decomp #556
Comments
code_80091750: Concerning D_802BFB80 typedef struct
{
u8 pixel_index_array[0x1000];
} struct_D_802BFB80;
extern struct_D_802BFB80 D_802BFB80[][2][8]; to typedef struct
{
u8 pixel_index_array[0x2000];
} struct_D_802BFB80;
extern struct_D_802BFB80 D_802BFB80[][2][4]; can lead to matches in the function(s) where its relevant. typedef struct {
u8 pixel_index_array[0x2000];
} thing0;
typedef struct {
u8 pixel_index_array[0x1000];
} thing1;
typedef union {
thing0 blah0[][2][4];
thing1 blah1[][2][8];
} the_union;
extern the_union D_802BFB80; If it makes stuff match, then great. It'd be super fugly, but if it works it works. |
PR'd func_8007C4A4 https://decomp.me/scratch/aJbM5 and func_80095AE0 https://decomp.me/scratch/77riB PR #618 PR #623 Matched func_80092290.s |
Matching: |
100% Match not necessary. Func Equivallency is good too. Don't forget to click "Family" in-case there are cleaner matches as some matches may be very fake-matchy.
render_objects
mov v1,v0
and some register allocation differencesplayer
variable is weird. The instructions are correct but out of placecode_8006E9C0
update_objects
D_800E5DF4
in some capacitycode_800AF9B0
save
code_80057C60
Vehicles/AI pathing
code_80005FD0
if
statement. There's also an unuseddouble
, don't know if its part of this function or the one after itmenu_items
textColor
function variable. The permuter reports an improvement if you make itvolatile
, but that seems wrong for several different reasons.menus.c
math_util.c
lenght
is screwycrash_screen.c
audio/external.c
select_current_sounds
from SM64, lots of work left to be doneaudio/load.c
audio/synthesis.c
process_envelope
from SM64. Diff is huge, no idea why. Speculation: the version of the ABI we have/use is horribly horribly wrong but is only noticeably wrong in this specific functionThe text was updated successfully, but these errors were encountered: