A couple minor quibbles

- Since the length is know the StringBuilder can be created with a size.

- sb.toString() is probably more efficient than new String(sb)

- I would like to see some IDN URL cases in the tests.

Mike

On Nov 1 2013, at 07:46 , Michael McMahon <michael.x.mcma...@oracle.com> wrote:

> Simple bug fix to new URLPermission class, caused by insufficient parameter 
> checking
> of the constructor.
> 
> webrev: http://cr.openjdk.java.net/~michaelm/8027687/webrev.1/
> 
> Thanks,
> Michael

Reply via email to