RFR: 8307508: IndirectVarHandle.isAccessModeSupported throws NPE [v3]

Chen Liang liach at openjdk.org
Mon Jun 12 23:23:09 UTC 2023


On Mon, 12 Jun 2023 22:54:08 GMT, Mandy Chung <mchung at openjdk.org> wrote:

>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Wrap lines that are too long
>
> 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.

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

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


More information about the core-libs-dev mailing list