Looks good! Thanks, Johnny Chen
On Sep 26, 2013, at 12:04 PM, Clemens Eisserer <[email protected]> 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
