Review request for JDK-8004727

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Tue Dec 18 10:02:28 PST 2012


Looks good!

  minor nits, Copyright incorrect in:
     Attribute.java, ClassWriter.java and others.....

ClassWriter.java

+        }
+        else
+            return 0;

other places in the file the convention is

} else
     return 0;


Kumar

> Hello,
>
> Please review the implementation of the javac side of method parameter
> reflection.  The feature request can be found here:
>
> http://bugs.sun.com/view_bug.do?bug_id=8004727
>
> Summary:
>
> The names, as well as other information about method parameters can now
> optionally be stored in Java class files.  There are updates to javac to
> create the parameter attributes when a -parameters flag is given.  There
> are also updates to the com.sun.tools.classfile API to correctly parse
> method parameter attributes, and to javap to print them.
>
> This code includes new tests, and has been tested by running numerous
> regression test suites, including some with javac hardwired to generate
> class files with parameter annotations.
>
>
> The open webrev can be found here:
>
> http://cr.openjdk.java.net/~coleenp/JDK-8004727/
>
>
> The specification for Method Parameter reflection can be found here:
>
> http://cr.openjdk.java.net/~abuckley/8misc.pdf
>
> Thanks,
> Eric




More information about the compiler-dev mailing list