Re: RFR 8167974 MethodHandles.iteratedLoop(...) fails with CCE in the case of iterating over array

[Claes Redestad]

+1

/Claes

On 2016-11-01 20:00, Paul Sandoz wrote:

On 31 Oct 2016, at 16:19, Claes Redestad <claes.redes...@oracle.com> 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.