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

Reflection - Confirm SqueakSource references still relevant #146

Open
dmitrizagidulin opened this issue Nov 2, 2015 · 1 comment
Open

Comments

@dmitrizagidulin
Copy link
Collaborator

The Using method wrappers to perform test coverage section has two SqueakSource references.

Method wrappers are a well-known technique for intercepting messages. In the
original implementation(http://www.squeaksource.com/MethodWrappers.html), a
method wrapper is an instance of a subclass of ==CompiledMethod==. When
installed, a method wrapper can perform special actions before or after invoking
the original method. When uninstalled, the original method is returned to its
rightful position in the method dictionary.

and

In Pharo, method wrappers can be implemented more easily by implementing
==run:with:in:== instead of by subclassing ==CompiledMethod==. In fact, there
exists a lightweight implementation of objects as method
wrappers(http://www.squeaksource.com/ObjectsAsMethodsWrap.html), but it is not
part of standard Pharo at the time of this writing.

Is this distinction (original implementation vs Pharo) still relevant? Perhaps update the links to SH?

@nicolaihess
Copy link
Collaborator

I think the distinction is still valid, and there is no (official pharo) package at SH.
(but neither MethodWrappers nor ObjectsAsMethods are working on Pharo)

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