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

use self reference rather than super #139

Merged
merged 1 commit into from
Jan 24, 2020
Merged

Conversation

JoranHonig
Copy link
Contributor

This change aligns the use of seek with the use in pop_frame
Additionally, inheritants can now override the behaviour of the seek call in seek_in_frame(...)

What was wrong?

The call to seek in seek_in_frame() was using super() to get a reference to the function seek().

How was it fixed?

super() -> self

Cute Animal Picture

Cute animal picture

This change aligns the use of seek with the use in pop_frame
Additionally, inheritants can now override the behaviour of the seek call in seek_in_frame(...)
@JoranHonig
Copy link
Contributor Author

@the reviewer of this pr
I thought about opening an issue for this change, but it was easier to just submit a PR right away.

Let me know if you would like a larger ratification for the change!

@davesque davesque self-requested a review January 24, 2020 17:33
Copy link
Contributor

@davesque davesque left a comment

Choose a reason for hiding this comment

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

LGTM. I think this was a little oversight on my part.

@davesque davesque requested a review from carver January 24, 2020 17:34
@carver
Copy link
Collaborator

carver commented Jan 24, 2020

@the reviewer of this pr

Poor Thomas Eichmann probably gets this a lot 😆

I thought about opening an issue for this change, but it was easier to just submit a PR right away.

Thanks for the PR!

LGTM. I think this was a little oversight on my part.

Yup, LGTM, too. Thanks for taking a look @davesque

@carver carver merged commit 15a96b1 into ethereum:master Jan 24, 2020
@carver carver mentioned this pull request Jan 24, 2020
3 tasks
carver added a commit to carver/eth-abi that referenced this pull request Jan 24, 2020
@carver
Copy link
Collaborator

carver commented Feb 27, 2020

@JoranHonig sorry it took a while to get this out. Thanks for the contribution! It's available in v2.1.1

fselmo pushed a commit to fselmo/eth-abi that referenced this pull request Apr 4, 2022
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