> On 31 Oct 2016, at 16:19, Claes Redestad <[email protected]> wrote:
> 
> Hi,
> 
> On 2016-10-31 22:47, Paul Sandoz wrote:
>> Hi,
>> 
>> And… 8167966 MethodHandles.iteratedLoop fails with IAE in the case of 
>> correct arguments.
>> 
>> The two issues are closely intertwined in terms of the fix. Please review:
>> 
>>   
>> http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8167974-mhs-iterated-loop-no-arrays/webrev/
> 
> I think this imposes a reasonable restriction and nicely cleans up the
> current specification. +1
> 
> I guess the if (!Iterable.class.isAssignableFrom(iterableType)) could
> be moved inside the preceding else without losing readability.
> 

I did that.

Thanks,
Paul.

Reply via email to