On 10/02/2014 07:00, michael cui wrote:
:
Please review the newest version at :
http://cr.openjdk.java.net/~tyan/michael/JDK-8028711/webrev.03/
This looks good to me.
There are a couple of places where javac wasn't updated. A few that I
noticed are:
test/com/sun/corba/5036554/TestCorbaBug.sh
test/sun/tools/jconsole/ResourceCheckTest.sh
test/sun/tools/native2ascii/resources/ImmutableResourceTest.sh
A minor comment on test/java/net/URLPermission/nstest/lookup.sh is that
the resulting line length is 182 characters and this will likely be
annoying for future side-by-side views. So I think I'd split this while
you are there.
-Alan.
