+1

-phil

On 12/4/19, 7:34 PM, Sergey Bylokhov wrote:
Any volunteers to review? =)

On 10/17/19 11:23 pm, Sergey Bylokhov wrote:
Hello.
Please review the fix for JDK 14.

Bug: https://bugs.openjdk.java.net/browse/JDK-8232226
Fix: http://cr.openjdk.java.net/~serb/8232226/webrev.00

This test creates a Color object on top of the SystemColor and checks
that the "equals()" will work properly for these two objects. But the test
uses Color(int RGB) constructor which skips the alpha channel of the
color. So the test fails if the SystemColor is transparent.

Actual change in the test is:
  - Color color = new Color(system.getRGB());
  + Color color = new Color(system.getRGB(), system.getAlpha() < 255);

But in the webrev it appears as a new test because it is moved from the closed repo.



Reply via email to