On 11/29/13 2:10 PM, Petr Pchelko wrote:
Hello, Sergey.
The fix looks good, just 2 tiny comments:
Big thanks! Will fix that.
line 64: you have a typo: shuould
line 65: testImages(images[1], images[1], false) - I believe this is also a
typo, you are comparing the same images, so the second 1 should be changed to 2.
With best regards. Petr.
On 28.11.2013, at 17:52, Alexander Scherbatiy <[email protected]>
wrote:
The fix looks good for me.
Thanks,
Alexandr.
On 11/28/2013 4:56 PM, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk 8.
The test is for "apple.awt.brushMetalLook" and "Window.style", which were added
in the JDK-7124513
Bug: https://bugs.openjdk.java.net/browse/JDK-8029010
Webrev can be found at: http://cr.openjdk.java.net/~serb/8029010/webrev.00
--
Best regards, Sergey.
