Webrev for making parts of TypeAnnotationPosition immutable

Eric McCorkle eric.mccorkle at oracle.com
Fri Jan 31 09:44:11 PST 2014


The following webrev implements a number of changes to
TypeAnnotationPosition.  It makes most of the core data immutable, and
changes the way that TypeAnnotationPositions are created.

As opposed to the zero-argument constructor with mutable fields,
TypeAnnotationPositions are now created using static methods.

This patch also marks some field and methods as deprecated (in
comments), as I plan to remove them in forthcoming patches.

One possibility that came up in review was making TypeAnnotationPosition
an abstract superclass, and having subclasses for each of the different
variants.

I also could not make some of the fields final (type_index,
exception_index, offset), as they are modified by Gen.  This is most
visible in the updated ClassReader.  I will be investigating ways to
deal with this in forthcoming patches.

The webrev is here:
http://cr.openjdk.java.net/~emc/8033004/webrev/

This patch has already been reviewed by the javac team; however, I would
like to give a chance for any tools (such as checkers), that currently
use type annotations to run tests before I integrate it.  Please look
over the patch, and run any tests that should be run.

Thanks,
Eric
-------------- next part --------------
A non-text attachment was scrubbed...
Name: eric_mccorkle.vcf
Type: text/x-vcard
Size: 303 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20140131/e5456211/eric_mccorkle.vcf 


More information about the compiler-dev mailing list