On Fri, 11 Feb 2022 15:24:45 GMT, Sam Brannen <[email protected]> wrote:
>> Joe Darcy has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Respond to review feedback.
>
> src/java.base/share/classes/sun/reflect/annotation/AnnotationInvocationHandler.java
> line 148:
>
>> 146: StringBuilder result = new StringBuilder(128);
>> 147: result.append('@');
>> 148: // Guard against shouldn't-happen NPE for a missing canonical
>> name
>
> NIT: A NPE would not be thrown. Rather, `"null"` would be appended to the
> buffer. Right?
Update comment in latest push.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7418
