Looks OK. I'll push once a 2nd set of eyes concurs.

-phil.

On 9/26/13 12:04 PM, Clemens Eisserer wrote:
Hi Phil,

the webrev and test need a little clean-up : adding the @test and @bug tag +
GPL
Please find a corrected version at
http://cr.openjdk.java.net/~ceisserer/8024895/webrev.01/
along with a (hopefully) jtreg conformant testcase at
http://cr.openjdk.java.net/~ceisserer/8024895/EABlitTest.java

Other than that the fix is fine although without looking at bit more broadly
I'm unsure how this cache works in a MT scenario.
It is guarded by the AWT-lock, but you're right - it is not benefitial
(but doesn't hurt either) under some circumstances.

Thanks, Clemens

Reply via email to