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