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

Jonathan Gibbons jjg at openjdk.java.net
Fri Sep 24 16:46:58 UTC 2021


On Fri, 24 Sep 2021 12:45:51 GMT, Pavel Rappo <prappo at openjdk.org> wrote:

>> Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address review comments;
>>   Move `JavadocTester.copyDir` to `ToolBox`; use FileVisitor code
>
> test/langtools/tools/lib/toolbox/ToolBox.java line 270:
> 
>> 268:     public void copyDir(Path fromDir, Path toDir) {
>> 269:         try {
>> 270:             Files.createDirectories(toDir);
> 
> Do not create this directory manually. It will be created first thing in your visitor.
> Suggestion:

The suggestion is incorrect.  The call is required to ensure that parent directories of `toDir` are created.   These are __not__ created by the call in the visitor.

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

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


More information about the javadoc-dev mailing list