Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicit invocant)

chromatic wrote:
>This proof of concept patch enables that, with $self as the invocant.  It 
>could use more robust testing, but all existing tests pass.

You should prototype this as a CPAN module, using the keyword hook.

>The trick of using lex_stuff_pvs() seems grotty, but it's far, far easier than 
>building a new optree with the appropriate lexical bindings within the method 
>production in perly.y.

As something to go in the core, I do not approve of this shortcut.
You should definitely build the optree rather than stuff source.

It also seems very unwise for the core to take the keyword "method" at
this stage.  We have several CPAN modules supplying "method" keywords
with various semantics, it's an unresolved area.

-zefram

• Proposal: "$->" for implicit invocant by Chia-liang Kao
• Re: Proposal: "$->" for implicit invocant by David Golden
• [PATCH] use feature 'method'; (was Re: Proposal: "$->" for implicit invocant) by chromatic
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicit invocant) by Zefram
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicit invocant) by Chris Prather
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" for implicit invocant) by chromatic
• [PATCH] use feature 'method' take two by chromatic
• Re: [PATCH] use feature 'method' take two by Zefram
• Re: [PATCH] use feature 'method' take two by Sam Vilain
• Re: [PATCH] use feature 'method' take two by Zefram
• Re: [PATCH] use feature 'method' take two by Sam Vilain
• Re: [PATCH] use feature 'method' take two by Reverend Chip
• Re: [PATCH] use feature 'method' take two by David E. Wheeler
• Re: [PATCH] use feature 'method' take two by Zefram
• Re: [PATCH] use feature 'method' take two by David Golden
• Re: [PATCH] use feature 'method' take two by David E. Wheeler
• Re: [PATCH] use feature 'method' take two by Matt Sergeant
• Re: [PATCH] use feature 'method' take two by chromatic
• Re: [PATCH] use feature 'method' take two by chromatic
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicit invocant) by Zefram
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicit invocant) by Eric Brine
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicit invocant) by Reini Urban
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicit invocant) by David Golden
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicitinvocant) by Steffen Mueller
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" for implicit invocant) by David E. Wheeler
• Re: [PATCH] use feature 'method'; (was Re: Proposal: "$->" forimplicit invocant) by David Golden
• Re: Proposal: "$->" for implicit invocant by Johan Vromans
• Re: Proposal: "$->" for implicit invocant by Zefram
• Re: Proposal: "$->" for implicit invocant by H.Merijn Brand