[8u] Approval request to backport the fix made for JDK-8129740

[Seán Coffey]

>> Does this bring behavioural changes of interest to JDK 8u for end 
>> users ? Anything needed in release note documentation ?
>>
>
> I don't think so. This is a straightforward defect fix.

Thanks for clarifying. Approved for jdk8u-dev.

Regards,
Sean.

On 25/05/2016 01:53, Srikanth wrote:
>
>
> On Tuesday 24 May 2016 07:58 PM, Seán Coffey wrote:
>>
>> JDK-8157142 is not a backport record of JDK-8129740. I suggest you 
>> close 8157142 as duplicate of 8129740. HGupdater will create a jdk8u 
>> backport record for you if you push your changeset with bug ID of 
>> 8129740.
>>
>
> My bad. Thanks for the instructions. I will implement it.
>>
>> Does this bring behavioural changes of interest to JDK 8u for end 
>> users ? Anything needed in release note documentation ?
>>
>
> I don't think so. This is a straightforward defect fix.
>
> Waiting for approval,
> Thanks
> Srikanth
>
>
>> Regards,
>> Sean.
>> On 24/05/2016 11:14, Srikanth wrote:
>>> Please approve.
>>>
>>> JDK9 defect: https://bugs.openjdk.java.net/browse/JDK-8129740
>>> JDK8 backport ticket: https://bugs.openjdk.java.net/browse/JDK-8157142
>>>
>>> (1) The original patch does not apply as is after path shuffling.
>>>
>>> However, manual application of the two failed hunks in 
>>> LambdaToMethod.java is straightforward
>>> and does not call for any real change in code.
>>>
>>> (2) The file 
>>> test/tools/javac/lambda/T8129740/SourceToSourceTranslationTest.java
>>> required some rewriting since the ToolBox class API is different 
>>> between
>>> JDK9 and JDK8.
>>>
>>> Relevant tests have been run and are all green.
>>>
>>> jdk8u webrev:
>>>
>>> http://cr.openjdk.java.net/~sadayapalam/JDK-8157142/webrev.00/
>>>
>>> 8u specific reviewer approval here:
>>>
>>> http://mail.openjdk.java.net/pipermail/compiler-dev/2016-May/010213.html 
>>>
>>>
>>> TIA,
>>> Srikanth
>>
>