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