Skip to content

MotorManager: Add class implementation. - #76

Merged
SwareJonge merged 1 commit into
doldecomp:mainfrom
cristian64:motormanager
Aug 25, 2026
Merged

MotorManager: Add class implementation.#76
SwareJonge merged 1 commit into
doldecomp:mainfrom
cristian64:motormanager

Conversation

@cristian64

Copy link
Copy Markdown
Contributor

No description provided.

@decomp-dev

decomp-dev Bot commented Aug 16, 2026

Copy link
Copy Markdown

Report for MarioClub_us (ec2544e - 8f20bdf)

📈 Matched code: 46.89% (+0.05%, +1544 bytes)
📈 Linked code: 41.72% (+0.06%, +1812 bytes)
📈 Matched data: 24.88% (+0.00%, +8 bytes)
📈 Linked data: 32.38% (+0.03%, +236 bytes)

✅ 8 new matches
Unit Item Bytes Before After
main/Kameda/MotorManager MotorManager::setMotor(MotorManager::MotorType, int) +472 0.84% 100.00%
main/Kameda/MotorManager MotorManager::setPowerMotor(int, float, unsigned char, unsigned char) +373 1.06% 100.00%
main/Kameda/MotorManager MotorManager::setAwardMotor(MotorManager::MotorType) +205 1.92% 100.00%
main/Kameda/MotorManager MotorManager::exec() +120 3.23% 100.00%
main/Kameda/MotorManager MotorManager::reset() +117 3.33% 100.00%
main/Kameda/MotorManager MotorManager::MotorManager(JKRHeap*) +85 38.21% 100.00%
main/Kameda/MotorManager MotorManager::~MotorManager() +17 84.19% 100.00%
main/Kameda/MotorManager .sdata2 +8 0.00% 100.00%
📈 1 improvement in an unmatched item
Unit Item Bytes Before After
main/Kameda/MotorManager .rodata +121 19.05% 98.67%

@cristian64
cristian64 force-pushed the motormanager branch 2 times, most recently from 6e6f51d to 7a4f112 Compare August 18, 2026 20:23
@cristian64
cristian64 marked this pull request as ready for review August 18, 2026 20:25
@cristian64

Copy link
Copy Markdown
Contributor Author

(Includes commit in #71; to be rebased.)

@cristian64

Copy link
Copy Markdown
Contributor Author

It has been rebased now.

@SwareJonge
SwareJonge merged commit dbdda50 into doldecomp:main Aug 25, 2026
1 check passed
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