RFR: 8303374: Implement JEP 455: Primitive Types in Patterns, instanceof, and switch (Preview) [v50]
Aggelos Biboudis
abimpoudis at openjdk.org
Thu Jan 25 13:53:06 UTC 2024
On Thu, 25 Jan 2024 09:57:13 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Aggelos Biboudis has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update year
>
> src/java.base/share/classes/java/lang/runtime/SwitchBootstraps.java line 148:
>
>> 146: * @throws NullPointerException if any argument is {@code null}
>> 147: * @throws IllegalArgumentException if any element in the labels array is null, if the
>> 148: * invocation type is not a method type of first parameter of a reference type,
>
> One thing we did often in the FFM API was to break up long `@throws` and split them into two or three `@throws` (javadoc allows the same exception to appear multiple times). Not saying you have to change this, just a possible suggestion (as I didn't know this!)
Indeed it looks better:
<img width="566" alt="Screenshot 2024-01-25 at 14 49 07" src="https://github.com/openjdk/jdk/assets/765983/84e6ae34-1e6b-4d36-98de-78f101be2336">
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15638#discussion_r1466406565
More information about the compiler-dev
mailing list