<AWT Dev> Review request for JDK-8157476 -Wlogical-not-parentheses warnings in JRSUIConstantSync.m
Sergey.Bylokhov at oracle.com
Thu Jun 2 19:11:45 UTC 2016
Note that this change should be pushed to jdk9-client ws:
On 31.05.16 21:35, Dan Smith wrote:
> Can I get somebody to look at this? Just point me in the right direction if I'm in the wrong place, please.
>> On May 20, 2016, at 1:52 PM, Dan Smith <daniel.smith at oracle.com> wrote:
>> 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.
Best regards, Sergey.
More information about the awt-dev