Hi, Eye-balled the changes, nothing obvious popped out. Looks good.
It's very minor, and should not block the code going back, but "occured" is a common misspelling of "occurred". Paul. On Mar 12, 2014, at 10:24 PM, Kumar Srinivasan <kumar.x.sriniva...@oracle.com> wrote: > Hello, > > Appreciate if someone cane review this change, so that I can push this into > the jdk9 repository, > refreshing the internal ASM sources from rev 1700 to 1721, of particular > interest is > Rev 1710, which fixes [1] known to cause verification errors in the VM. > Thereafter I will be back-porting this changeset into 8u20. > > http://cr.openjdk.java.net/~ksrini/8037221/ > > The following regression tests have been run: > nashorn/test > jdk/test/tools/pack200 > jdk/test/java/lang/invoke > jdk/test/java/util > jdk/test/javax/script > jdk/test/sun/tools/jrunscript > jdk/test/closed/com/oracle/jfr > langtools/test > > Nashorn additional test targets: > test > test262 > octane > > Thanks in advance > Kumar > > [1] > http://forge.ow2.org/tracker/?group_id=23&atid=100023&func=detail&aid=316545