Skip to content

Working more on zNPCFXCinematic with one 100% and started on other functions#519

Merged
mattbruv merged 25 commits intobfbbdecomp:mainfrom
Redberd36:main
Apr 18, 2025
Merged

Working more on zNPCFXCinematic with one 100% and started on other functions#519
mattbruv merged 25 commits intobfbbdecomp:mainfrom
Redberd36:main

Conversation

@Redberd36
Copy link
Contributor

No description provided.

@github-actions
Copy link

🆗 ✅

+494 bytes
🆗 Bot Rating: A decent contribution. Thank you!

No Regressions 🎉

Progress: 10

📈 main/SB/Game/zNPCFXCinematic - .text +439 bytes -> 19.03%
📈 main/SB/Game/zTalkBox - .text +36 bytes -> 0.39%
📈 main/SB/Game/zNPCFXCinematic - .sdata2 +18 bytes -> 29.46%
📈 main/SB/Game/zNPCFXCinematic - NCIN_HazProjShoot(const zCutsceneMgr*, NCINEntry*, int) +196 bytes -> 44.14%
📈 main/SB/Game/zNPCFXCinematic - NCIN_HazTTSteam_AR(const zCutsceneMgr*, NCINEntry*, RpAtomic*, RwMatrixTag*, unsigned int, unsigned int) +4 bytes -> 1.11%
📈 main/SB/Game/zNPCFXCinematic - NCIN_HammerStreak_AR(const zCutsceneMgr*, NCINEntry*, RpAtomic*, RwMatrixTag*, unsigned int, unsigned int) +4 bytes -> 1.15%
📈 main/SB/Game/zNPCFXCinematic - NCIN_HazTTSteam_Upd(const zCutsceneMgr*, NCINEntry*, int) +4 bytes -> 1.89%
main/SB/Game/zNPCFXCinematic - NCIN_HammerStreak_Upd(const zCutsceneMgr*, NCINEntry*, int) +132 bytes -> 100.00%
📈 main/SB/Game/zTalkBox - ztalkbox::reset_all() +36 bytes -> 28.12%
main/SB/Game/zNPCFXCinematic - NCIN_WaterSplash(const zCutsceneMgr*, NCINEntry*, int) +100 bytes -> 100.00%

}

switch (fxrec->typ_ncinfx)
switch(fxrec->typ_ncinfx)
Copy link
Member

Choose a reason for hiding this comment

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

Can you run the formatter on this file so things like this don't sneak into PRs?

Comment on lines +16 to +30
void ztalkbox::reset_all()
{
volatile int* sharedMemory = reinterpret_cast<volatile int*>(ztalkbox_shared.active);

int value = 0;
*sharedMemory = value;
*(sharedMemory + 8) = value;
*(sharedMemory + 9) = 0;
*(sharedMemory + 10) = 0;
*(sharedMemory + 11) = 0;
*(sharedMemory + 12) = 0;

clear_layout(); //FIXME: Find and replace with xtextbox::layout::clear()
return;
}
Copy link
Member

Choose a reason for hiding this comment

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

This is something I wouldn't want to include, it's very rough and only raises the function to 28%, so it's a big work in progress still, and doesn't necessarily give us anything useful to work off of. I'd remove it for now from this PR.

@github-actions
Copy link

⚠️ 🔥

+458 bytes
🆗 Bot Rating: A decent contribution. Thank you!

Regressions: 19

⚠️ main/SB/Game/zNPCTypeBossSandy - .data -4 bytes -> 54.29%
⚠️ main/SB/Game/zNPCTypeBossSandy - .rodata -24 bytes -> 35.59%
⚠️ main/SB/Game/zNPCTypeBossSandy - .sdata2 -15 bytes -> 36.28%
⚠️ main/SB/Game/zNPCTypeBossSandy - .text -1704 bytes -> 17.31%
⚠️ main/SB/Core/x/xCM - .sbss -8 bytes -> 0.00%
⚠️ main/SB/Core/x/xCM - .sdata2 -14 bytes -> 0.00%
⚠️ main/SB/Core/x/xCM - .text -440 bytes -> 0.00%
⚠️ main/SB/Game/zNPCTypeBossSandy - sitCB(xGoal*, void*, en_trantype*, float, void*) -268 bytes -> 0.00%
⚠️ main/SB/Game/zNPCTypeBossSandy - getUpCB(xGoal*, void*, en_trantype*, float, void*) -388 bytes -> 0.00%
⚠️ main/SB/Game/zNPCTypeBossSandy - runToRopeCB(xGoal*, void*, en_trantype*, float, void*) -240 bytes -> 0.00%
⚠️ main/SB/Game/zNPCTypeBossSandy - clotheslineCB(xGoal*, void*, en_trantype*, float, void*) -260 bytes -> 0.00%
⚠️ main/SB/Game/zNPCTypeBossSandy - zNPCGoalBossSandyIdle::Enter(float, void*) -164 bytes -> 8.89%
⚠️ main/SB/Game/zNPCTypeBossSandy - zNPCGoalBossSandyIdle::Process(en_trantype*, float, void*, xScene*) -300 bytes -> 0.00%
⚠️ main/SB/Game/zNPCTypeBossSandy - zNPCGoalBossSandyIdle::Exit(float, void*) -84 bytes -> 0.00%
⚠️ main/SB/Core/x/xCM - xCMprep(xCreditsData*) -240 bytes -> 0.00%
⚠️ main/SB/Core/x/xCM - xCMupdate(float) -80 bytes -> 0.00%
⚠️ main/SB/Core/x/xCM - xCMrender() -40 bytes -> 0.00%
⚠️ main/SB/Core/x/xCM - xCMstart(xCreditsData*, float, xBase*) -40 bytes -> 0.00%
⚠️ main/SB/Core/x/xCM - xCMstop() -40 bytes -> 0.00%

Progress: 8

📈 main/SB/Game/zNPCFXCinematic - .text +439 bytes -> 19.03%
📈 main/SB/Game/zNPCFXCinematic - .sdata2 +18 bytes -> 29.46%
📈 main/SB/Game/zNPCFXCinematic - NCIN_HazProjShoot(const zCutsceneMgr*, NCINEntry*, int) +196 bytes -> 44.14%
📈 main/SB/Game/zNPCFXCinematic - NCIN_HazTTSteam_AR(const zCutsceneMgr*, NCINEntry*, RpAtomic*, RwMatrixTag*, unsigned int, unsigned int) +4 bytes -> 1.11%
📈 main/SB/Game/zNPCFXCinematic - NCIN_HammerStreak_AR(const zCutsceneMgr*, NCINEntry*, RpAtomic*, RwMatrixTag*, unsigned int, unsigned int) +4 bytes -> 1.15%
📈 main/SB/Game/zNPCFXCinematic - NCIN_HazTTSteam_Upd(const zCutsceneMgr*, NCINEntry*, int) +4 bytes -> 1.89%
main/SB/Game/zNPCFXCinematic - NCIN_HammerStreak_Upd(const zCutsceneMgr*, NCINEntry*, int) +132 bytes -> 100.00%
main/SB/Game/zNPCFXCinematic - NCIN_WaterSplash(const zCutsceneMgr*, NCINEntry*, int) +100 bytes -> 100.00%

@mattbruv mattbruv merged commit ac232d6 into bfbbdecomp:main Apr 18, 2025
5 checks passed
JoshSanch added a commit that referenced this pull request May 8, 2025
* ship it, YOLO

* test

* BossSB2 functions and mem offsets

* wip push

* push fix

* couple of bossplankton functions.

* znpcmessenger function fix

* More xMath3 work (#513)

* More xMath3 work

* This isnt matching

* Update 🆗 bot

* zNPCGoalAmbient: zNPCGoalJellyBumped Near 100% Match (#514)

* zNPCTypeCommon: Add declaration for zNPCCommon::HaveLOSToPos

* zNPCTypeAmbient: Add declarations for zNPCJellly ActLikeOctopus and JellyBoneWorldPos

* zFX: Add overload declarations for zFX_SpawnBubbleTrail

* xFX: Add overload declaration for xFXStreakUpdate

* zNPCGoalAmbient: Nearly 100% match for all zNPCGoalJellyBumped functions

* zNPCGoalAmbient: Fix flipped registers in CalcEndPoint implementation

* Robo-Sandy: Function Progress (#515)

* zNPCTypeVillager: Limit scope of g_sndTrax_Villager

* zNPCTypeBossSandy: Add SandyLimbSpring struct from DWARF data

* zNPCTypeBossSandy: Unextern floats and match most data

* Sandy: Several function matches + some WIP

* Sandy: Fix regression in ZNPC_AnimTable_BossSandy and increase matchingness

* zNPCFXCinematic -> NPARMgmt::KillAll() 100% (#516)

* Fix zNPCFXCinematic implementation

* All NPCCone functions 100%

* forgot one change.

* Moved Static S32 init to the top of file in zNPCFXCinematic.cpp

* Removed commented code in zNPCFXCinematic.cpp

* Removed commented code at the bottom of zNPCFXCinematic.cpp

* Removed commented code at the top of zNPCFXCinematic.h

* Added parameter names for UVSliceSet, UVBaseSet, ColorSet

* Removed the blank git file

* xSnd work (#517)

* 40% - 92% for some NCIN_Par functions in zNPCFXCinematic (#518)

* Fix zNPCFXCinematic implementation

* All NPCCone functions 100%

* forgot one change.

* Moved Static S32 init to the top of file in zNPCFXCinematic.cpp

* Removed commented code in zNPCFXCinematic.cpp

* Removed commented code at the bottom of zNPCFXCinematic.cpp

* Removed commented code at the top of zNPCFXCinematic.h

* Added parameter names for UVSliceSet, UVBaseSet, ColorSet

* Removed the blank git file

* 40% - 92% for some NCIN_Par functions in zNPCFXCinematic

* Pointer name fixes for NCIN_Par functions in zNPCFXCinematic + one 100% zTalkBox function

* Deleted static variables top of script

* xCM work (#520)

* Robo-Sandy Callback Funcs and Idle Goal Impl (#521)

* zNPCTypeBossSandy: Add SandyLimbSpring struct from DWARF data

* zNPCTypeBossSandy: sit, getUp, runToRope, clothesline callbacks

* zNPCTypeBossSandy: zNPCGoalBossSandyIdle virtual implementations

* Fix: Remove redef of SandyLimbSpring bc rebasing is hard

* Working more on zNPCFXCinematic with one 100% and started on other functions (#519)

* Fix zNPCFXCinematic implementation

* All NPCCone functions 100%

* forgot one change.

* Moved Static S32 init to the top of file in zNPCFXCinematic.cpp

* Removed commented code in zNPCFXCinematic.cpp

* Removed commented code at the bottom of zNPCFXCinematic.cpp

* Removed commented code at the top of zNPCFXCinematic.h

* Added parameter names for UVSliceSet, UVBaseSet, ColorSet

* Removed the blank git file

* 40% - 92% for some NCIN_Par functions in zNPCFXCinematic

* Pointer name fixes for NCIN_Par functions in zNPCFXCinematic + one 100% zTalkBox function

* Deleted static variables top of script

* Working more on zNPCFXCinematic with one 100% and started some others.

* Working more on zNPCFXCinematic with one 100% and started some others.

* Fixed a minor error in zNPCFXCinematic

* Fixed a minor error in zTalkBox

* Fixed another minor error in zTalkBox

* Fixed formatting and removed a zTalkBox function

* Match most of iMix (#522)

* Match most of iMix

* Correct MIXUnMute signature

* Hack around not being able to use _AXVPB. Having issues including it for some reason.

* Robosandy: Progress (#523)

* zNPCTypeBossSandy: Add SandyLimbSpring struct from DWARF data

* Fix: Remove redef of SandyLimbSpring bc rebasing is hard

* zNPCTypeBossSandy: Taunt, Chase, and Melee goal virtual method impls

* zNPCTypeBossSandy: NoHead Goal Enter match, Process WIP

* zEntHangable progress (#524)

* first fn

* save

* moreo prrogress

* static vars

* hangable_reset

* setfollow

* setupfx

* stub fns

* cleanup

* fix build

* zCamera: 95% match on zCameraFlyUpdate (#525)

* zNPCSupplement work (#526)

* zCamera: Equiv/Near Equiv for more functions (#527)

* zNPCSupport work (#528)

* zCamera: zCameraFlyRestoreBackup and zCameraRewardUpdate functions (#529)

* zNPCSpawner: Matches and additional function definitions (#530)

* zNPCSpawner: Matches and additional function definitions

* Config: Use more VS Code-friendly settings

* Little bit of zNPCtypeBossPlankton

---------

Co-authored-by: Matt P <mattp3259@gmail.com>
Co-authored-by: escape209 <agtescape@gmail.com>
Co-authored-by: mattbruv <5638426+mattbruv@users.noreply.github.com>
Co-authored-by: Joshua Sanchez <joshuaesanch@gmail.com>
Co-authored-by: Red <53205920+Redberd36@users.noreply.github.com>
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