-
Notifications
You must be signed in to change notification settings - Fork 41
Hide certain symbols for X360 COFFs #248
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
Conversation
update: have spotted |
Wouldnt it be better to handle this via config? #238 |
I'd have to disagree - except_datas aren't actual functions, they're 8 byte structs containing pointers to exception data. Since our decomped code doesn't automatically spawn the same except data structs, I'd argue it's not worth keeping them in the function list window. As for unwinds and catches....I still think those would be better off automatically hidden - your only real use of matching these 100% would be if you're trying to link, which X360 isn't currently doing. But that said, I'm open to further discussion about this. |
I'm not arguing against hiding these things, I understood that it's data, I mean i think the ignore lists should be adjustable via the project not hardcoded into objdiff |
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.
Are we sure there's no automatic way to detect these? e.g. certain symbol type/metadata
objdiff-core/config-schema.json
Outdated
"description": "Display pooled data references in functions as fake relocations." | ||
}, | ||
{ | ||
"id": "ppc.showUnwinds", |
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.
Is this functionally different than clicking "Show hidden symbols" in Diff Options?
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.
Functionally, no it is not - I suppose instead of this option we can just bundle in the except_datas and the __unwinds/__catches under the "hidden symbols" umbrella?
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.
Yes, and in fact that's what you're doing with SymbolFlag::Hidden
, so it should work already!
This hides any symbol starting with
except_data_
automatically if you're working on X360.In addition,
__unwind
s are now hidden by default, with the option to show them again added in Arch Settings.fixes #232