On Tue, 8 Jul 2025 14:56:23 GMT, Khalid Boulanouare <[email protected]> wrote:

>> Sets setOpaque for lightweight, and uses a compares colors with tolerance in 
>> MacOS X machines.
>
> Khalid Boulanouare has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Adds extra space after if
>   
>   Co-authored-by: Andrey Turbanov <[email protected]>

Please change copyright years for test as well

test/jdk/java/awt/Mixing/AWT_Mixing/MixingPanelsResizing.java line 304:

> 302:     }//fail()
> 303:     private static boolean isAlmostEqualColor(Color color, Color 
> refColor) {
> 304:         final int COLOR_TOLERANCE_MACOSX = 10;

I think this field should go with the other final variables at the top for 
visibility

-------------

PR Review: https://git.openjdk.org/jdk/pull/26166#pullrequestreview-3078084729
PR Review Comment: https://git.openjdk.org/jdk/pull/26166#discussion_r2247209974

Reply via email to