[OpenJDK 2D-Dev] Review request for http://cr.openjdk.java.net/~ceisserer/fix10/

Phil Race philip.race at oracle.com
Thu Sep 26 22:49:08 UTC 2013


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




More information about the 2d-dev mailing list