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

22695-Make-value-for-work-method-after #2012

Conversation

MarcusDenker
Copy link
Member

@MarcusDenker
Copy link
Member Author

This PR fixed #value for #after on MethodNode.

The strategy is to not wrap in a block, but instead use the method wrapper functionality that we need for primitive method anyway.

  • Now #after on method wraps method instead of doing block wrapping

  • wrapper always saves return value in a temp

TODO: ensure: wrapping, which will be added later back

@MarcusDenker MarcusDenker merged commit 9a2bec8 into pharo-project:development Nov 27, 2018
@MarcusDenker MarcusDenker deleted the 22695-Make-value-for-work-method-after branch February 2, 2022 12:13
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.

None yet

1 participant