Skip to content

pygccxml fails in enum/typedef hash function #70

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

Closed
dermont123 opened this issue Feb 16, 2017 · 5 comments
Closed

pygccxml fails in enum/typedef hash function #70

dermont123 opened this issue Feb 16, 2017 · 5 comments
Assignees
Labels
Milestone

Comments

@dermont123
Copy link

` File "./../pygccxml/declarations/enumeration.py", line 51, in hash
return super(enumeration_t, self).hash(self)
TypeError: hash() takes exactly 1 argument (2 given)

def __hash__(self):
    return super(enumeration_t, self).__hash__()
   -->  #return super(enumeration_t, self).__hash__(self)

def __hash__(self):
    return super(typedef_t, self).__hash__()
    --> #return super(typedef_t, self).__hash__(self)

`
hash.zip

@iMichka
Copy link
Collaborator

iMichka commented Feb 16, 2017

Hi. Thanks for reporting this.

I had already fixed this some time ago, but only on the develop branch (8db77ca). I saw this when running pygccxml with pypy.

I will backport the patch and create a 1.8.5 release this weekend.

@iMichka iMichka self-assigned this Feb 16, 2017
@iMichka iMichka added the bug label Feb 16, 2017
@iMichka iMichka added this to the 1.8.5 milestone Feb 16, 2017
@iMichka
Copy link
Collaborator

iMichka commented Feb 19, 2017

I created the 1.8.5 release branch that can be used for testing purposes:
https://github.com/gccxml/pygccxml/tree/hotfix/v1.8.5

I am waiting on another patch to be tested first, and as soon as this is done, I'll make the release. It may take 1 or 2 days before I have time to make the release though.

@dermont123
Copy link
Author

Ok I'll try it out, but the changes for the typdef/enum were ones I already tested so I know it already resolves the problem.
return super(enumeration_t, self).hash()
return super(typedef_t, self).hash()

The above don't appear to be on the devlop branch.

@iMichka
Copy link
Collaborator

iMichka commented Feb 20, 2017

Yes, it will once the 1.8.5 branch is merged to develop too.

@iMichka
Copy link
Collaborator

iMichka commented Feb 22, 2017

I just released 1.8.5, which was uploaded to pypi too. The changes are in the develop branch too. Closing here. Thanks for reporting again, and for your patience.

@iMichka iMichka closed this as completed Feb 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants