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
