'Re: 8242292: java.nio.file.FileSystems.getFileSystem(URI) should throw IAE if the URI's scheme is nu'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-nio-dev
Subject:    Re: 8242292: java.nio.file.FileSystems.getFileSystem(URI) should throw IAE if the URI's scheme is nu
From:       Brian Burkhalter <brian.burkhalter () oracle ! com>
Date:       2020-04-08 20:34:32
Message-ID: D11ECE62-2E31-4A3C-BA64-483C80BF7E4F () oracle ! com
[Download RAW message or body]

> On Apr 8, 2020, at 1:23 PM, Alan Bateman <Alan.Bateman@oracle.com> wrote:
> 
> On 08/04/2020 16:45, Brian Burkhalter wrote:
> > 
> > Please see http://cr.openjdk.java.net/~bpb/8242292/webrev.01/ \
> > <http://cr.openjdk.java.net/~bpb/8242292/webrev.01/>. Test job in progress. 
> I assume the "throws URISyntaxException" isn't needed, otherwise looks okay to me.

Apparently not; webrev.01 update in place.

Thanks,

Brian


[Attachment #3 (unknown)]

<html><head><meta http-equiv="Content-Type" content="text/html; \
charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: \
space; line-break: after-white-space;" class=""><br class=""><div><br \
class=""><blockquote type="cite" class=""><div class="">On Apr 8, 2020, at 1:23 PM, \
Alan Bateman &lt;<a href="mailto:Alan.Bateman@oracle.com" \
class="">Alan.Bateman@oracle.com</a>&gt; wrote:</div><br \
class="Apple-interchange-newline"><div class=""><span style="caret-color: rgb(0, 0, \
0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: \
normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: \
0px; text-transform: none; white-space: normal; word-spacing: 0px; \
-webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline \
!important;" class="">On 08/04/2020 16:45, Brian Burkhalter wrote:</span><br \
style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; \
font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: \
normal; text-align: start; text-indent: 0px; text-transform: none; white-space: \
normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" \
class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; \
font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: \
normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; \
white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; \
-webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br class="">Please \
see<span class="Apple-converted-space">&nbsp;</span><a \
href="http://cr.openjdk.java.net/~bpb/8242292/webrev.01/" \
class="">http://cr.openjdk.java.net/~bpb/8242292/webrev.01/</a>. Test job in \
progress.<br class=""><br class=""></blockquote><span style="caret-color: rgb(0, 0, \
0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: \
normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: \
0px; text-transform: none; white-space: normal; word-spacing: 0px; \
-webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline \
!important;" class="">I assume the "throws URISyntaxException" isn't needed, \
otherwise looks okay to me.</span></div></blockquote></div><br class=""><div \
class="">Apparently not; webrev.01 update in place.</div><div class=""><br \
class=""></div><div class="">Thanks,</div><div class=""><br class=""></div><div \
class="">Brian</div></body></html>



[prev in list] [next in list] [prev in thread] [next in thread]
Configure | About | News | Add a list | Sponsored by KoreLogic