Return method name from memoize #22
Conversation
| end | ||
|
|
||
| macro memoize def g; end | ||
| end |
There was a problem hiding this comment.
- It can be tested like
expect(G.memoize(:g)).to eq :g - Also via class-reader and
@memoize_result = memoize def d, I guess.
There was a problem hiding this comment.
Good call, but i want to avoid G modifications in specs.
I.e. i could use F, but when i call memoize on F, it will have F#m memoized
There was a problem hiding this comment.
but i want to avoid G modifications in specs.
define_method is modification, but OK.
I.e. i could use F, but when i call memoize on F, it will have F#m memoized
That's why stabbing consts is better then global definition.
There was a problem hiding this comment.
define_method is modification, but OK.
Is is called before suite
That's why stabbing consts is better then global definition.
This PR does not point specs refactoring, but i agree with that a lot
There was a problem hiding this comment.
This PR does not point specs refactoring
I know, I know, just side notes.
|
Thanks for PR! |
Closes #20