... thanks a lot, Peter and Vladimir! Best,
Michael > Am 09.11.2015 um 13:42 schrieb Vladimir Ivanov <vladimir.x.iva...@oracle.com>: > > Looks good! > > Best regards, > Vladimir Ivanov > > On 11/9/15 3:16 PM, Peter Levart wrote: >> Hi all, >> >> Thanks for analysis, reviews and discussion that hopefully beheaded this >> hydra. >> >> I added an assert to method setSpeciesDataToConcreteBMHClass() that >> verifies the presence of @Stable annotation on the SPECIES_DATA field of >> the generated class. It caught a bug I made when I specified a binary >> class name "java/lang/invoke/Stable" in FieldVisitor.visitAnnotation() >> instead of type signature "Ljava/lang/invoke/Stable;" >> >> The consequence was not drastic - the field was just not being >> interpreted by VM as @Stable. >> >> With that adjustment and successful re-run of jtreg tests, I ask for >> re-confirmation to push the following: >> >> http://cr.openjdk.java.net/~plevart/jdk9-dev/BMH.race/webrev.07/ >> >> Regards, Peter >> >> On 11/06/2015 01:31 PM, Vladimir Ivanov wrote: >>> Peter, >>> >>>> >>>> http://cr.openjdk.java.net/~plevart/jdk9-dev/BMH.race/webrev.06/ >>>> >>> >>> Looks really good! Reviewed. >>> >>>> - the SPECIES_DATA field in generated class can not be static final when >>>> it is initialized out of <clinit>. Just static. It was just static >>>> before when it was initialized in <clinit> and could be static final. We >>>> can make it @Stable at least. >>>> >>> Not sure it is on hot path, but I'm all for marking the field as @Stable >>> >>>> - there's no need for empty <clinit> method now. Removed. >>>> >>>> Basic java/lang/invoke jtreg tests pass. >>> >>> Best regards, >>> Vladimir Ivanov >> -- <http://www.oracle.com/> Dr. Michael Haupt | Principal Member of Technical Staff Phone: +49 331 200 7277 | Fax: +49 331 200 7561 Oracle Java Platform Group | LangTools Team | Nashorn Oracle Deutschland B.V. & Co. KG, Schiffbauergasse 14 | 14467 Potsdam, Germany <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment