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

Properties with opname "LOAD_DEREF" #9

Closed
Bapt5 opened this issue Dec 5, 2022 · 5 comments
Closed

Properties with opname "LOAD_DEREF" #9

Bapt5 opened this issue Dec 5, 2022 · 5 comments

Comments

@Bapt5
Copy link

Bapt5 commented Dec 5, 2022

Hi
I'm using autoslot in a project. But I don't understand why your librairy don't work on some of my class. I tried to debug in your module, and I found that the the opname is equal to LOAD_DEREF instead of LOAD_FAST. Do you know why it do that? And how can I fix it?

@cjrh
Copy link
Owner

cjrh commented Dec 6, 2022

Hi @Bapt5

Thanks for posting. If you can post a short example here that exhibits the problem, I can take a look at it this coming weekend? The core issue is either a simple oversight on my part, or a change in the opcodes from when I first made autoslot. Either way, it should be quite easy to fix once we have a short example to test it on.

@Bapt5
Copy link
Author

Bapt5 commented Dec 7, 2022

Hi @cjrh
It's on the Discussion class of pronotepy

@cjrh
Copy link
Owner

cjrh commented Dec 22, 2022

Ok I have a test that reproduces. Will fix it shortly.

@cjrh cjrh closed this as completed in 5b896b8 Dec 22, 2022
cjrh added a commit that referenced this issue Dec 22, 2022
Handle `self` obtained via LOAD_DEREF not LOAD_FAST. Fixes #9
@cjrh
Copy link
Owner

cjrh commented Dec 22, 2022

@Bapt5 I added the feature and pushed a new release to PyPI. Hopefully it works for you. thanks again for reporting the issue :)

@Bapt5
Copy link
Author

Bapt5 commented Dec 22, 2022

Thanks a lot

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

2 participants