> Another solution is to exclude this file from HEADLESS compilation.
> I am not sure how to achieve it. Any suggestion?
I suppose that is possible and I expect we can do that
See in the make file, where I think you just need to add entries to
LIBAWT_HEADLESS_EXCLUDES := medialib
although I have not tried it.
Hmm .. I wonder why medialib needs to be explicitly excluded
from headless ? .. but that is for another day.
I have another question: why do you mention only OGLBlitLoops.c ?
I've flicked through a number of the C files in the same location and
all look to have the same issue.
-phil.
On 04/21/2016 06:33 AM, Ajit Ghaisas wrote:
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 ..
[Ajit ] Yes. The Solaris compiler still complains about this declaration in
code. Hence, I have fixed the reported warnings after removing the suppression
from makefile.
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 ??
[Ajit] : I fixed warnings reported for 'maybe-uninitialized' and
'E_DECLARATION_IN_CODE' warning types in two .c files in webrev.
There was no warning after removal of 'int-to-pointer-cast' suppression from
makefile. No code change was made for this type of warning.
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.
[Ajit] : Nope. Moving jni.h or jlong.h inclusions outside #ifndef HEADLESS did
not help. We still get E_EMPTY_TRANSLATION_UNIT warning.
To get rid of this warning, there are suggestions to make a typedef - and not
use it anywhere - but, I would rather keep the suppression in makefile than
defining a typedef without actual usage.
Another solution is to exclude this file from HEADLESS compilation. I am not
sure how to achieve it. Any suggestion?
-phil.
Request you to review following webrev :
http://cr.openjdk.java.net/~aghaisas/8074829/webrev.00/
Regards,
Ajit
