RFR: 8334332: TestIOException.java fails if run by root [v2]

SendaoYan syan at openjdk.org
Tue Jun 18 06:43:10 UTC 2024


On Tue, 18 Jun 2024 06:19:23 GMT, Andrey Turbanov <aturbanov at openjdk.org> 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 56:
> 
>> 54:     public static void main(String... args) throws Exception {
>> 55:         var tester = new TestIOException();
>> 56:         if(Platform.isRoot() && !tester.isWindows()) {
> 
> Suggestion:
> 
>         if (Platform.isRoot() && !tester.isWindows()) {

Thanks for the review. The code has been changed according to the review opinions of @pavelrappo.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19731#discussion_r1643905043


More information about the javadoc-dev mailing list