[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