[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