https://bugs.openjdk.java.net/browse/JDK-8157476

I noticed this code triggering a LOT of warnings when building under Xcode 7.3 
in Mac OS X, and the fix is trivial.

(I'm guessing this native file belongs to AWT, but please redirect me if I'm 
wrong.)

-----

diff -r 8c75ff8185c6 src/java.desktop/macosx/native/libosxui/JRSUIConstantSync.m
--- a/src/java.desktop/macosx/native/libosxui/JRSUIConstantSync.m       Fri May 
20 11:12:02 2016 -0700
+++ b/src/java.desktop/macosx/native/libosxui/JRSUIConstantSync.m       Fri May 
20 13:41:53 2016 -0600
@@ -90,7 +90,7 @@
     apple_laf_JRSUIConstants_ ## clazz ## __ ## constant
 
 #define CONSTANT_CHECK(clazz, constant)                                \
-    JRS_CONSTANT(clazz, constant) == JNI_CONSTANT(clazz, constant)
+    ( JRS_CONSTANT(clazz, constant) == JNI_CONSTANT(clazz, constant) )
 
 #define CONSISTENCY_CHECK(clazz, constant)                            \
     if ( !CONSTANT_CHECK(clazz, constant) ) return NO;

-----

If that looks okay, I'm happy to push this myself.  As noted in the bug 
comments, I think this is a legitimate bug fix, though -- so if somebody wants 
to write a test exposing the bug, you may just want to assign to yourself and 
take responsibility for pushing.

Thanks,
Dan

Reply via email to