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

diff for RELs #54

Open
robojumper opened this issue May 23, 2024 · 0 comments
Open

diff for RELs #54

robojumper opened this issue May 23, 2024 · 0 comments

Comments

@robojumper
Copy link

It would be nice to have something like ninja diff working for RELs. A use case can be found in https://github.com/robojumper/ss-dtk/tree/rel-wip (SOUE01) - with the latest commit on that branch, objdiff shows no difference, but checksum verification for the d_t_tackle REL fails for not immediately obvious reasons.

The reason in this case is that two virtual function overrides were (accidentally) swapped in the symbols, which resulted in an incorrect vtable after linking, but to figure this out I had to binary diff the whole REL in a hex editor - which is doable for a small REL like this but scales poorly.

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

No branches or pull requests

1 participant