backport JDK-8130506 to 8 ?

[Remi Forax]

Thanks Srikanth.

Rémi

----- Mail original -----
> De: "Srikanth" <srikanth.adayapalam at oracle.com>
> À: compiler-dev at openjdk.java.net
> Envoyé: Lundi 18 Janvier 2016 05:40:36
> Objet: Re: backport JDK-8130506 to 8 ?
> 
> The fix is straightforward and applies cleanly after path shuffling. The
> workaround of having
> to use inner classes in lieu of lambda seems painful relative to the
> ease of backport.
> 
> I'll organize this.
> 
> Thanks!
> Srikanth
> 
> On Sunday 17 January 2016 03:00 AM, Maurizio Cimadamore wrote:
> > I will leave the definitive answer to Srikanth, but from the looks of
> > it, the patch seems trivial, so it doesn't seem crazy to backport it.
> >
> > Maurizio
> >
> > On 16/01/16 17:47, Remi Forax wrote:
> >> answsers inlined,
> >>
> >> ----- Mail original -----
> >>> De: "Jonathan Gibbons" <jonathan.gibbons at oracle.com>
> >>> À: compiler-dev at openjdk.java.net
> >>> Envoyé: Vendredi 15 Janvier 2016 21:52:36
> >>> Objet: Re: backport JDK-8130506 to 8 ?
> >>>
> >>> Rémi,
> >> Hi Jon,
> >>
> >>> There's maybe 3 parts to this:
> >>>
> >>> 1. Is it conceptually reasonable to backport the fix
> >> You can not do a methodHandle.invoke() inside a lambda in 8, so it's
> >> reasonable to backport the fix.
> >> Now, there are not a lot of people that uses java.lang.invoke API and
> >> the fact that the workaround is easy,
> >> replace the lambda by an inner class, backporting the fix may not
> >> worth it.
> >>
> >>> 2. Is it a relatively easy backport (i.e. does the fix for 9 apply
> >>> reasonably
> >>> easily, or do we need a different fix, specific to 8)
> >> I believe the patch applies cleanly (minus the line number).
> >>
> >>> 3. Making it happen
> >>>
> >>> If folk can weigh in on 1 and 2, and the answer to both questions is
> >>> "yes",
> >>> we can make 3 happen easily enough.
> >>>
> >>> -- Jon
> >> Rémi
> >>
> >>>
> >>>
> >>> On 01/15/2016 12:03 PM, Remi Forax wrote:
> >>>> Hi guys,
> >>>> the code I use to demonstrate how to create your own dynamically typed
> >>>> language on top of the JVM now triggers JDK-8130506 [1] :(
> >>>> i think the bug could have appear last year but last year none of my
> >>>> students was using IntelliJ IDEA, so the bug stay hidden.
> >>>>
> >>>> I wonder if the corresponding patch could be backported to 8, or maybe
> >>>> Attila and me are the only one to call MethodHandle.invoke inside a
> >>>> lambda
> >>>> ?
> >>>>
> >>>> cheers,
> >>>> Rémi
> >>>>
> >>>> [1] https://bugs.openjdk.java.net/browse/JDK-8130506
> >>>
> >
> 
>