RFR: 8004699 : Add type annotation storage to Constructor, Field and Method (original) (raw)

Peter Levart peter.levart at gmail.com
Mon Dec 17 16:00:46 UTC 2012

Hi Joel,

82 // This is set by the vm at Method creation 83 private byte[] type_annotations;

Wouldn't it be better to initialize this field in the constructor? You could create an overloaded constructor if you wanted to be compatible with previous versions of the VM.

Regards, Peter

On 12/17/2012 04:41 PM, Joel Borggrén-Franck wrote:

Here is the first in a series of changes to add type annotation support to reflection.

http://cr.openjdk.java.net/~jfranck/8004699/webrev.00/ While this adds overhead all instances of Field, Constructor, and Method we need this change in order to coordinate with the changes going in to HotSpot There is a plan to remove the additional overhead by consolidating all annotation fields, though this will have to wait a bit. cheers /Joel

More information about the core-libs-dev mailing list