-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Decompile skidoo #1733
base: develop
Are you sure you want to change the base?
Decompile skidoo #1733
Conversation
8e5e0fa
to
5ce797c
Compare
Download the built assets for this pull request: |
src/tr2/decomp/skidoo.c
Outdated
fx->object_id = O_EXPLOSION; | ||
} | ||
|
||
Effect_ExplodingDeath(g_Lara.skidoo, ~SKIDOO_GUN_MESH, 0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks – the value indeed was wrong, rather than ~4 it should've been -4 or ~(4 - 1). I have pushed a correction, but I'm unable to test it at the moment and would appreciate info if it was enough to fix the issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW the name SKIDOO_GUN_MESH
I think is wrong and should be changed – but I don't fully understand its usage. For sure it has to do with the track mesh, but I'm not certain as to why track_mesh & 4
yields a truthy value for armed snowmobiles and a falsy value for normal snowmobiles.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I don't follow it either TBH. Will need some studying.
It failed to recognize function pointer arguments.
5ce797c
to
6a6872e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There's a desync in demo 3 - the skidoo misses a small jump 3 seconds into the video. Latest dev snapshot: Video.1.mp4PR: Video.2.mp4 |
Checklist
Description
#1691