> On 11 May 2016, at 18:31, shilpi.rast...@oracle.com wrote: > > Hi All, > > Please review the updated webrev- > http://cr.openjdk.java.net/~srastogi/8149574/webrev.07/ >
1219 FieldVisitor fv; 1220 MethodVisitor mv; 1221 AnnotationVisitor av0; Field “fv is not used. Since “av0” is only used once, might as well declare it at line #1246. Can you break up the long lines at #1242 & #1252 ? My inclination is to turn the anon static block into a method returning byte[] and then do: /* <JAVA DOC explaining the class that is generated rather than just floating as is the current case> */ private static final byte[] T_BYTES = generateT(); private static byte[] generateT() { … } One concern, more so because of my ignorance, is why can the default package be used. Does anyone know? Paul. > Changed the anonymous class package with no package name. > > Regards, > Shilpi >