<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div><br></div><div>On 10 Aug 2015, at 18:52, Brian Burkhalter <<a href="mailto:brian.burkhalter@oracle.com">brian.burkhalter@oracle.com</a>> wrote:<br><br></div><blockquote type="cite"><div><meta http-equiv="Content-Type" content="text/html charset=windows-1252">I have modified the test as suggested. The updated version is here:<div><br></div><div><a href="http://cr.openjdk.java.net/~bpb/8132497/webrev.01/">http://cr.openjdk.java.net/~bpb/8132497/webrev.01/</a></div></div></blockquote><div><br></div><div>Looks good.</div><div><br></div><div>-Chris</div><div><br></div><br><blockquote type="cite"><div><div>Thanks,</div><div><br></div><div>Brian</div><div><br><div><div>On Aug 10, 2015, at 9:54 AM, Chris Hegarty <<a href="mailto:chris.hegarty@oracle.com">chris.hegarty@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;">This looks ok to me Brian, and consistent with newFileSystem(Path, ClassLoader).</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;">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.</span></blockquote></div><br></div></div></blockquote></body></html>