[9] RFR of 8132497: (fs) FileSystems.newFileSystem(URI, ..) doesn't handle UOE thrown by provider
Chris Hegarty
chris.hegarty at oracle.com
Mon Aug 10 19:38:45 UTC 2015
> On 10 Aug 2015, at 18:52, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:
>
> I have modified the test as suggested. The updated version is here:
>
> http://cr.openjdk.java.net/~bpb/8132497/webrev.01/
Looks good.
-Chris
> Thanks,
>
> Brian
>
>> On Aug 10, 2015, at 9:54 AM, Chris Hegarty <chris.hegarty at oracle.com> wrote:
>>
>> This looks ok to me Brian, and consistent with newFileSystem(Path, ClassLoader).
>>
>> The only minor comment I have is about delete(path) in the test. It could use the utility method from the test library, FileUtils. deleteFileWithRetry(Path), to avoid issues with deletion on Windows.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20150810/232b8f3e/attachment.html>
More information about the nio-dev
mailing list