Skip to content

Mobjdrv partially done#13

Open
Ilwyd wants to merge 27 commits into
doldecomp:mainfrom
Ilwyd:mobjdrv
Open

Mobjdrv partially done#13
Ilwyd wants to merge 27 commits into
doldecomp:mainfrom
Ilwyd:mobjdrv

Conversation

@Ilwyd

@Ilwyd Ilwyd commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

I did as much of this as I was able to, but I think some (mobjCalcMtx, mobjHitEntry) are beyond my ability at the moment.

@decomp-dev

decomp-dev Bot commented Jun 14, 2026

Copy link
Copy Markdown

Report for G8MJ01 (e372143 - 688c821)

📈 Matched code: 11.25% (+0.04%, +1496 bytes)

✅ 9 new matches
Unit Item Bytes Before After
MarioSt/driver/mobjdrv mobjCalcMtx2 +280 0.00% 100.00%
MarioSt/driver/mobjdrv mobjDelete +264 0.00% 100.00%
MarioSt/driver/mobjdrv mobjSetPosition +264 0.00% 100.00%
MarioSt/driver/mobjdrv mobjNameToPtrNoAssert +204 0.00% 100.00%
MarioSt/driver/mobjdrv mobjNameToPtr +188 0.00% 100.00%
MarioSt/driver/mobjdrv mobjRunEvent +128 0.00% 100.00%
MarioSt/driver/mobjdrv mobjCheckExec +112 0.00% 100.00%
MarioSt/driver/mobjdrv mobjHitObjPtrToPtr +28 0.00% 100.00%
MarioSt/driver/mobjdrv mobjGetHint +28 0.00% 100.00%
📈 5 improvements in unmatched items
Unit Item Bytes Before After
MarioSt/driver/mobjdrv mobjNearDistCheck2 +671 0.00% 99.92%
MarioSt/driver/mobjdrv mobjEntry +475 0.00% 99.90%
MarioSt/driver/mobjdrv .rodata +180 0.00% 70.62%
MarioSt/driver/mobjdrv mobjCheckItemboxOpen +163 0.00% 99.90%
MarioSt/driver/mobjdrv .sdata2 +15 10.00% 29.17%
📉 1 regression in an unmatched item
Unit Item Bytes Before After
MarioSt/driver/mobjdrv mobjHitEntry -35 2.51% 0.00%

Comment thread src/driver/mobjdrv.c Outdated
Comment thread src/driver/mobjdrv.c Outdated
Comment thread src/driver/mobjdrv.c Outdated
Comment thread src/driver/mobjdrv.c Outdated
Comment thread src/driver/mobjdrv.c Outdated
Comment thread src/driver/mobjdrv.c
Comment thread src/driver/mobjdrv.c Outdated
Comment thread src/driver/mobjdrv.c Outdated
Comment thread src/driver/mobjdrv.c Outdated
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