<AWT Dev> [9] Review Request: 8129894 NSApplicationAWT.m:343:72: error: comparison of constant 777 with expression of type 'NSEventSubtype'
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Thu Jul 9 10:49:27 UTC 2015
Hello.
Please review the fix for jdk9.
This is the last issue, which prevents us to build the jdk on osx using
the latest clang w/o --disable-warnings-as-errors.
This code was integrated in
http://cr.openjdk.java.net/~anashaty/8068886/webrev.00. The new clang
complains that our constant is not related to NSEventSubtype. But
actually this is not a problem, because our custom events are
NSApplicationDefined and subtype is "a numeric identifier that further
differentiates custom events"[1].
To supress the warning we can change constant 777 to some value from
NSEventSubtype. + small cleanup. I also tried to split the long lines
but in this case the change became much bigger, because there are a
number of such places.
[1]
https://developer.apple.com/library/mac/documentation/Cocoa/Reference/ApplicationKit/Classes/NSEvent_Class/
Bug: https://bugs.openjdk.java.net/browse/JDK-8129894
Webrev can be found at: http://cr.openjdk.java.net/~serb/8129894/webrev.02
--
Best regards, Sergey.
More information about the awt-dev
mailing list