RFR: JDK-8013225: Refresh jdk's private ASM to the latest. (original) (raw)

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Mon Apr 29 22:25:31 UTC 2013

The restyling changes obfustucated things a bit but I didn't see anything of concern in casual review.

I had hoped to see the updated SmallSet that didn't try to implement Iterator directly.

It looks the SmallSet needs more discussion.. barring that anyone else have any other concerns with this change ? If I don't hear any objections I will push this on 05/01.

Thanks Kumar

Looks OK. The testing, which you have done, is the important qualifier for this change. Mike On Apr 25 2013, at 13:24 , Kumar Srinivasan wrote:

Here is the webrev: http://cr.openjdk.java.net/~ksrini/8013225/webrev.0/

Thanks Kumar

Hello,

Please review changes which essentially contains asm5future in asm's mainline repository, I have tested this patch with Nashorn (so has Sundar), as well as Lambda's usage. Fixes that I know of: 1. supports v52.0 class files and JSR 308/Type Annotations changes Thanks to Remi 2. elimination of "" usages in variable names 3. several javadoc changes and one reported by Sundar http://jbs.oracle.com/bugs/browse/JDK-8010083

Thanks Kumar

More information about the core-libs-dev mailing list