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

Missing tags #487

Merged
merged 1 commit into from
Aug 16, 2023
Merged

Missing tags #487

merged 1 commit into from
Aug 16, 2023

Conversation

sevaa
Copy link
Contributor

@sevaa sevaa commented Aug 14, 2023

Addresses #484

@KShingala
Copy link

Dont think just adding enums is enough, I get the below error when I try you change,

elftools.construct.adapters.MappingError: no decoding mapping for 46 [tag]

@sevaa
Copy link
Contributor Author

sevaa commented Aug 15, 2023

From here it's impossible to tell what are you doing and what's the exact nature of the error. Who wrote backend_z3.py - is that your script, or from some third party package? If the latter, why don't you report the issue to its authors and see if they can debug it?

Also, sorry for stating the obvious, we don't know how exactly did you apply this fix on your side. That, too, can be done correctly or incorrectly, and you are not telling.

Copy link
Owner

@eliben eliben left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks straightforward, thanks

@eliben eliben merged commit cac2f1c into eliben:master Aug 16, 2023
3 checks passed
@sevaa sevaa deleted the fix_missing_tags_v3 branch September 6, 2023 13:36
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.

3 participants