On Wed, 19 May 2021 10:47:46 GMT, Pavel Rappo <pra...@openjdk.org> 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

Reply via email to