Skip to content

Some RPSystem TU's - #148

Open
ThePlayerRolo wants to merge 2 commits into
doldecomp:masterfrom
ThePlayerRolo:master
Open

Some RPSystem TU's#148
ThePlayerRolo wants to merge 2 commits into
doldecomp:masterfrom
ThePlayerRolo:master

Conversation

@ThePlayerRolo

Copy link
Copy Markdown
Contributor

Currently RPSysQueue is linked with RP_80186E40 being merged with RPSysLoadScene.

I tried to define RP_80186E40 as a seperate tu but since the lbl for it is never used anywhere it gets optimized out, leading to linking errors.

I assume its apart of RPSysLoadScene due to its position in text, however if you think otherwise I can undo it

@decomp-dev

decomp-dev Bot commented Aug 29, 2026

Copy link
Copy Markdown

Report for RSPE01_01 (eb5969b - 4e92789)

📈 Matched code: 33.97% (+0.01%, +304 bytes)
📈 Linked code: 28.10% (+0.01%, +188 bytes)
📈 Matched data: 37.55% (+0.00%, +4 bytes)
📈 Linked data: 54.07% (+0.00%, +24 bytes)

✅ 6 new matches
Unit Item Bytes Before After
main/Pack/RPSystem/RPSysQueuedScene RPSysQueuedScene::CreateInstance(EGG::Heap*) +112 0.00% 100.00%
main/Pack/RPSystem/RPSysLoadScene __sinit_\RPSysLoadScene_cpp +88 0.00% 100.00%
main/Pack/RPSystem/RPSysQueuedScene RPSysQueuedScene::~RPSysQueuedScene() +64 0.00% 100.00%
main/Pack/RPSystem/RPSysLoadScene __arraydtor$25785 +28 0.00% 100.00%
main/Pack/RPSystem/RPSysQueuedScene RPSysQueuedScene::reset() +12 0.00% 100.00%
main/Pack/RPSystem/RPSysLoadScene .ctors +4 0.00% 100.00%
📈 3 improvements in unmatched items
Unit Item Bytes Before After
main/Pack/RPSystem/RPSysLoadScene .bss +60 0.00% 93.75%
main/Pack/RPSystem/RPSysQueuedScene .data +13 0.00% 85.71%
main/Pack/RPSystem/RPSysQueuedScene .sbss +5 0.00% 66.67%

@kiwi515

kiwi515 commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

hi, thank you- i appreciate the pr! im pretty confident the static initializer is from some mystery tu because it only exists in revision 1 of the game, despite RPSysLoadScene being basically equivalent between the two revisions. also, there arent any symbols in wii fit u that suggest there could be extra data

@@ -0,0 +1,21 @@
#include <RPSystem.h>

RP_SINGLETON_IMPL_EX(RPSysQueuedScene)

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.

Suggested change
RP_SINGLETON_IMPL_EX(RPSysQueuedScene)
RP_SINGLETON_IMPL_EX(RPSysQueuedScene);

@@ -0,0 +1,21 @@
#include <RPSystem.h>

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.

Suggested change
#include <RPSystem.h>
#include <Pack/RPSystem.h>

reset();
}

void RPSysQueuedScene::reset() {

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.

could u copy paste the doc-comment from above the declaration pls

*/
void reset();

inline s32 getSceneID() {

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.

Suggested change
inline s32 getSceneID() {
s32 getSceneID() const {

*/
class RPSysQueuedScene {
RP_SINGLETON_DECL_EX(RPSysQueuedScene);

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.

these functions will be private by default without public:

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