RFR: 8310849: Pattern matching for instanceof and arrayType cleanup in j.l.invoke and j.l.reflect [v3]
Chen Liang
liach at openjdk.org
Mon Jun 26 22:02:29 UTC 2023
On Mon, 26 Jun 2023 20:54:42 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> simplify equals by replacing if with and operators
>
> src/java.base/share/classes/java/lang/invoke/MemberName.java line 995:
>
>> 993: return result;
>> 994: ReflectiveOperationException ex = result.makeAccessException();
>> 995: if (ex instanceof IllegalAccessException iae) throw iae;
>
> Suggestion:
>
> if (ex instanceof IllegalAccessException iae) throw iae;
Don't know why, but sometimes JDK devs leaves 2 spaces for certain breaks (like Joe Darcy tends to leave 2 for sentence breaks in Javadocs). Can anyone explain why? In MemberName in particular, I see a mix of 2 and 1 spaces for single-line if statements, so I will take your suggestion.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14642#discussion_r1242852444
More information about the core-libs-dev
mailing list