Match Jetski::HandleClick
#1380
Conversation
| m_unk0x160 = ((DuneBuggy::GetColorOffset(g_varJSWNSHY5) * 5 + 15) * 2); | ||
| m_unk0x160 += DuneBuggy::GetColorOffset(g_varJSFRNTY5); | ||
| MxS32 local8 = DuneBuggy::GetColorOffset(g_varJSWNSHY5); | ||
| m_unk0x160 = 10 * (local8 + 3); |
There was a problem hiding this comment.
You can use the constants from the actions header here. I believe it's the base offset 40 + local8*10 iirc. The compiler will change the math to the same assembly
There was a problem hiding this comment.
You mean to use JetskiScript::c_Jetski_Actor for the 3? Not sure what you have in mind for the 10.
There was a problem hiding this comment.
Try IsleScript::c_JetskiFuelMeter + (local8 - 1) * 10 (untested)
There was a problem hiding this comment.
If you look at the constants in that header it should become clearer
There was a problem hiding this comment.
Thanks for the hint! While your suggestion matches LEGO1, it does not match BETA10, unfortunately. I have commented what the code does and also cleaned things up a little. While I was at it, I also (roughly) decompiled the BETA10 version - it still has one issue which may be entropy related.
Closes #1368. The BETA10 match cannot be improved significantly since it has been changed a lot.