On Mon, 12 Jun 2023 23:17:03 GMT, Chen Liang <li...@openjdk.org> wrote:

>> test/jdk/java/lang/invoke/VarHandles/IndirectVarHandleTest.java line 44:
>> 
>>> 42:         var lookup = MethodHandles.lookup();
>>> 43:         var intArrayVh = 
>>> MethodHandles.arrayElementVarHandle(int[].class);
>>> 44:         var addOne = lookup.bind((IntUnaryOperator) a -> a + 1, 
>>> "applyAsInt", MethodType.methodType(int.class, int.class));
>> 
>> Nit: a couple of long lines (L44, 48 and 55) can be wrapped into separate 
>> line.
>
> Done in a second push. Also wrapped the summary line (which was too long) 
> from your suggestion patch.

Looks good.  thanks.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14409#discussion_r1227349773

Reply via email to