On Wed, 6 Oct 2021 10:39:19 GMT, Pavel Rappo <[email protected]> wrote:

>> Jonathan Gibbons has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   address review comments
>
> test/langtools/jdk/javadoc/tool/CheckManPageOptions.java line 95:
> 
>> 93:         toolDocletOnly.removeAll(MISSING_IN_MAN_PAGE);
>> 94:         if (!toolDocletOnly.isEmpty()) {
>> 95:             error(" The following options are defined by the tool or 
>> doclet, but not defined in the man page:\n"
> 
> Remove stray whitespace:
> Suggestion:
> 
>             error("The following options are defined by the tool or doclet, 
> but not defined in the man page:\n"

👍

> test/langtools/jdk/javadoc/tool/CheckManPageOptions.java line 287:
> 
>> 285:         return list;
>> 286:     }
>> 287:  }
> 
> Add trailing newline.

👍

-------------

PR: https://git.openjdk.java.net/jdk/pull/5833

Reply via email to