[9] Review Request: 8079965 Stop ignoring warnings for libawt_lwawt

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Mon Sep 14 14:06:42 UTC 2015


Hello.
Please review the fix for jdk9.

In the fix I remove WARNINGS_AS_ERRORS_clang option from the 
libawt_lwawt library, and fix some of the issues:

- jlong_md.h:69:9: warning: 'ptr_to_jlong' macro redefined. This is 
because the "jni_util.h" and 
"JavaNativeFoundation.framework/Headers/JNFJNI.h" both define this 
macro. I cleared our headers to eliminate this warning.

- PrinterView.m:207:21: warning: implicit conversion from enumeration 
type 'NSPaperOrientation' (aka 'enum NSPaperOrientation') to different 
enumeration type 'NSPrintingOrientation'. The problem is that the Apple 
changed the returned type of [NSPrintInfo orientation] from 
NSPrintingOrientation to NSPaperOrientation. Note that the 
NSPaperOrientation is available since OSX 10.9, which means that this 
change break the build on 10.8. Is it acceptable or should I suppress 
this warning? [1]

- CGraphicsDevice.m:336:41: warning: comparison between pointer and 
integer ('void *' and 'jint' (aka 'int')) if ([screenID pointerValue] == 
displayID). I have changed the type from pointerValue to unsignedIntValue.

Also I added "enum-conversion" to the DISABLED_WARNINGS_clang to 
suppress some warnings to fix them later, because it should be 
investigated how to fix it properly (ImageSurfaceData.m:1090:93: 
warning: implicit conversion from enumeration type 'CGImageAlphaInfo' 
(aka 'enum CGImageAlphaInfo') to different enumeration type 'CGBitmapInfo')

After the fix all new warnings will break the build. The currently 
disabled warnings will be fixed as part of JDK-8074825 [2].

jprt build passed.

[1] 
https://developer.apple.com/library/mac/releasenotes/General/APIDiffsMacOSX10_9/AppKit.html
[2] https://bugs.openjdk.java.net/browse/JDK-8074825


Bug: https://bugs.openjdk.java.net/browse/JDK-8079965
Webrev: http://cr.openjdk.java.net/~serb/8079965/webrev.01

-- 
Best regards, Sergey.



More information about the build-dev mailing list