RFR: 8340087: (fs) Files.copy include symbolic link detailed message for NoSuchFileException [v2]

xpbob duke at openjdk.org
Fri Sep 20 11:52:35 UTC 2024


On Fri, 13 Sep 2024 07:19:11 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> xpbob has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Reimport package
>
> src/java.base/unix/classes/sun/nio/fs/UnixFileSystem.java line 1013:
> 
>> 1011:                     if (symbolicLink != null) {
>> 1012:                         throw new NoSuchFileException(source.toString(), null, "due to " + symbolicLink);
>> 1013:                     }
> 
> The default provider implementation can't use Files.* methods, otherwise you end up with recursive usage.
> 
> In any case, I think the starting out to decide if anything should change as the existing NoSuchFileException is not wrong.

@AlanBateman @bplb 
sorry for late,Thanks for guiding the process

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20984#discussion_r1768462153


More information about the nio-dev mailing list