On 04/20/2016 12:27 PM, Sergey Bylokhov wrote:
2d-dev added.

In fact all these are 2D. No AWT warnings here.



I am not sure but why "declaration in the code" is a bad thing and we should fix it?
- DISABLED_WARNINGS_solstudio := E_DECLARATION_IN_CODE

I cannot find the documentation in solaris studio for this warning.

I don't mind fixing it if it is still an issue but does the current compiler actually complain about it ?
The SS11 -> SS12 upgrade might have got a more modern C compiler ..

On 20.04.16 11:57, Ajit Ghaisas wrote:
Hi,

     Bug : https://bugs.openjdk.java.net/browse/JDK-8074829
This bug is to remove warning suppressions from makefile and fix the warnings for libawt_headless library.

I have removed following warning suppressions & fixed the warnings for libawt_headless library.
     DISABLED_WARNINGS_gcc := maybe-uninitialized int-to-pointer-cast

What made that one go away ??

     DISABLED_WARNINGS_solstudio := E_DECLARATION_IN_CODE

     Warning suppression that cannot be removed :
     DISABLED_WARNINGS_solstudio := E_EMPTY_TRANSLATION_UNIT
This is due to the fact that - jdk/src/java.desktop/share/native/common/java2d/opengl/OGLBlitLoops.c file becomes empty file in case of HEADLESS mode compilation.

Sigh .. there ought to be "informational" warnings as well as "risky practice" warnings and
this should be in the former category.
You could move something like the jni.h and jlong.h imports outside to see if that shuts it up.
Not saying that is what we want to do but it would be interesting to check.

-phil.

     Request you to review following webrev :
     http://cr.openjdk.java.net/~aghaisas/8074829/webrev.00/

Regards,
Ajit




Reply via email to