On Mon, 17 Jun 2024 15:10:52 GMT, Pavel Rappo <[email protected]> wrote:
>> SendaoYan has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> delete an extra whitespace
>
> test/langtools/jdk/javadoc/doclet/testIOException/TestIOException.java line
> 58:
>
>> 56: if(Platform.isRoot() && !tester.isWindows()) {
>> 57: throw new SkippedException("root user has privileged will
>> make this test fail.");
>> 58: }
>
> Suggestion:
>
> if (Platform.isRoot() && !tester.isWindows()) {
> throw new SkippedException("root user has privileges that will
> make this test fail");
> }
> var tester = new TestIOException();
`isWindows()` is not static, so call this function need a instance
`TestIOException` object, thus `var tester = new TestIOException();` should at
before line 56.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19731#discussion_r1643003292
