Proposed patch for review

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Mon Oct 24 02:39:53 PDT 2011


On 24/10/11 06:25, Jonathan Gibbons wrote:
> I've posted a patch from our newest contributor, Vicente Zaldivar.
> The patch is a cleanup to the JCTree.tag values, to use enums
> instead of small integers.
>
> You can see the patch here:
> http://cr.openjdk.java.net/~jjg/6921494/webrev.00/
>
> Thanks to Vicente for the contribution. Review feedback here or
> to Vicente, please.
>
> -- Jon
Great work - couple of comments below:

*) JCTree - I would add an import static as follows:

import static com.sun.tools.javac.tree.JCTree.Tag.*

This way you will get rid of all the qualified identifier (i.e. most of 
the code in JCTree should go back to its original form).

The same applies to all files (I listed the files where there is at 
least more than a single qualified ident):

-) Annotate.java
-) Attr.java
-) Check.java
-) Flow.java
-) Lower.java
-) MemberEnter.java
-) Resolve.java (a non-static import of JCTree.Tag would do fine)
-) Gen.java
-) JavacElements.java
-) JavacParser.java
-) Pretty.java
-) TreeInfo.java
-) TreeMaker.java (a non-static import of JCTree.Tag would do fine)
-) ClassDocImpl.java

Maurizio



More information about the compiler-dev mailing list