RFR: 8004699 : Add type annotation storage to Constructor, Field and Method

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