RFR(S): 8154754: MethodHandles.countedLoop errors in deriving loop arguments, result type, and local state

[Claes Redestad]

Hi Michael,

thanks for simplifying and cleaning things up. With both patches applied 
the code looks good to me.

/Claes

On 2016-04-22 14:41, Michael Haupt wrote:
> Hi Claes,
>
> thanks for this one as well; see http://cr.openjdk.java.net/~mhaupt/8154754/webrev.01/ for an updated webrev. This patch still depends on the one for 8154751.
>
> Best,
>
> Michael
>
>> Am 22.04.2016 um 12:18 schrieb Claes Redestad <claes.redestad at oracle.com>:
>>
>> Hi,
>>
>> looks good to me. resultType == void.class ? zero(void.class) : identity(resultType) appears twice and unconditionally used at least once, thus could be profitably extracted to a variable for readability/imaginary performance gain. Thanks! /Claes
>>
>> On 2016-04-20 15:46, Michael Haupt wrote:
>>> Dear all,
>>>
>>> please review this change. It depends on the one about 8154751 posted earlier [1].
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8154754
>>> Webrev: http://cr.openjdk.java.net/~mhaupt/8154754/webrev.00/
>>>
>>> Thanks,
>>>
>>> Michael
>>>
>>> [1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-April/040386.html
>>>