Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Support lambda/proc in advice method matchers #11

Open
gcao opened this Issue Dec 31, 2012 · 2 comments

Comments

Projects
None yet
2 participants
Owner

gcao commented Dec 31, 2012

Example:

before lambda{target.methods}, :do_this
before proc{target.methods}, :do_this

Owner

gcao commented Dec 31, 2012

We support "target{methods}" already and it is pretty handy. What is the benefit of supporting lambda?

Owner

gcao commented Dec 31, 2012

target{methods} and deferred logic are hard to understand. Wrapping target.methods in a lambda is more explicit.

@mensfeld mensfeld added the feature label Jul 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment