Skip to content
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

WIP of decompiling Snake18 #2039

Draft
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

DarrenSweeney
Copy link
Contributor

Right now just focusing on getting files decompiled and less so on code correctness/cleanup.

…sing on getting files decompiled and less so on code correctness/cleanup.
@DarrenSweeney
Copy link
Contributor Author

I'm not sure if the file path is correct, should snake18.c be inside animal/liquid or? Previous pull request mentioned those paths being pullic knowlege from the PC port but I don't know where to find those.

@DarrenSweeney DarrenSweeney changed the title Start of decompiling Snake18, decompiled 2 files. Start of decompiling Snake18 Nov 10, 2024
@DarrenSweeney
Copy link
Contributor Author

Ah made a small mistake, will fix.

@DarrenSweeney
Copy link
Contributor Author

Made some more progress reversering more functions. Going to keep this pull request open and try to keep reversering more from time to time, code needs a cleanup pass.

@nameloCmaS
Copy link

nameloCmaS commented Jan 2, 2025

I have made a small contribution and worked on Snake18_800CB3E8 but not sure how I can contribute it as it would rely on this PR being merged.

You can pick the decompiled function from the below link if simpler.

https://decomp.me/scratch/2GhuB

@DarrenSweeney
Copy link
Contributor Author

@nameloCmaS Thanks! Super helpful, I'm back at my computer from the holidays so going to start integrating the solved decomp links.

@DarrenSweeney
Copy link
Contributor Author

@nameloCmaS I hade to shuffel one line from your decomp to get it match 100%: https://decomp.me/scratch/l8vKC. Simple change. Your link shows it matches 91%

…ely make a snake18_2.c so I can submit a chunk of reversed functions.
@DarrenSweeney DarrenSweeney changed the title Start of decompiling Snake18 WIP of decompiling Snake18 Jan 6, 2025
…en't figured out the other funcitons. Contributions from @nameloCmaS
@DarrenSweeney
Copy link
Contributor Author

Made more progress, reversed some more functions putting it at 19 functions reversed. Quite a bit more to go.

@DarrenSweeney DarrenSweeney marked this pull request as draft January 6, 2025 20:45
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