RFR JDK-4880778

Alan Bateman Alan.Bateman at oracle.com
Sat Feb 2 08:29:42 PST 2013


On 02/02/2013 14:39, John Zavgren wrote:
> Greetings:
>
> Please consider the following modification of our classes for handling URLs.
>
> http://cr.openjdk.java.net/~jzavgren/4880778/webrev.01/
>
>
This looks okay to me.

For the URLStreamHandler setURL methods then an alternative to removing 
the statement would be just adjust the wording, say replace "be able to 
call the set method on a URL" to "be able to use this method to set the 
values of the URL fields".

Minor nit, but there are couple of alignment issues. In URL then the 
arguments spill onto 2 or 3 lines so they can be re-aligned. In 
URLStreamHandler it look slike the setURL method isn't quite lined up 
with the other methods (pre-dates your proposed changes).

-Alan.



More information about the net-dev mailing list