On Wed, 19 May 2021 10:47:46 GMT, Pavel Rappo <[email protected]> wrote:
>> Ian Graves has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Typos
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/tool/Messager.java line 71:
>
>> 69: if (!(instance instanceof Messager messenger))
>> 70: throw new InternalError("no messager instance!");
>> 71: return messenger;
>
> Every project has quirks. Out of many quirks of JavaDoc, we love this one the
> most. Perhaps you misread that type's name: it's Messager [sic!]. Please
> rename the variable either to `messager` or to a shorter one.
@pavelrappo let's consider changing this name sometime (not now)
-------------
PR: https://git.openjdk.java.net/jdk/pull/4105