RFR: JDK-8274172: Convert JavadocTester to use NIO [v3]

Jonathan Gibbons jjg at openjdk.java.net
Fri Sep 24 17:07:07 UTC 2021


On Fri, 24 Sep 2021 17:00:47 GMT, Pavel Rappo <prappo at openjdk.org> wrote:

>> Jonathan Gibbons has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains four commits:
>> 
>>  - Simplify ToolBox.copyDir
>>    Revert bad IDE suggestion
>>  - Merge with upstream/master
>>  - Address review comments;
>>    Move `JavadocTester.copyDir` to `ToolBox`; use FileVisitor code
>>  - JDK-8274172: Convert JavadocTester to use NIO
>
> test/langtools/tools/lib/toolbox/ToolBox.java line 277:
> 
>> 275:             if (toDir.getParent() != null) {
>> 276:                 Files.createDirectories(toDir.getParent());
>> 277:             }
> 
> This is exactly what I was going to re-suggest after I read your reply on my initial suggestion. We're on the same page. Thanks for simplifying this stuff.

Actually, we could simplify it to an unconditional  `Files.createDirectories(toDir.toAbsolutePath().getParent())`

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

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


More information about the javadoc-dev mailing list