On 19/06/2013 15:23, Peter Levart wrote:
Hi,

Since the bulk of changes to annotations and reflection have stabilized, I'm bringing up a re-based batch that I have proposed some months ago:

http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-February/014203.html

:

This patch is one angle of attack for bug 7122142. The other is, off course, removing the other synchronized keyword (on Class.initAnnotationsIfNecessary()). I'm planning to do that too, but in a more straight-forward manner.

Regards, Peter

Thank you for coming back to this.

I've looked over the webrev and the approach looks good to me. Joel might want to look at this too. Do you think you could include a test (as we try to include a test with all fixes if we can)?

It would be good to remove the synchronizaiton on initAnnotationsIfNecessary too, but one step as time (and smaller changes are always easier to discuss).

-Alan


Reply via email to