RFR: JDK-8148629: Disable remaining warnings in awt/fontmanager
Erik Joelsson
erik.joelsson at oracle.com
Fri Feb 5 09:05:55 UTC 2016
A while back, Magnus did a tremendous job of disabling most warnings in
the JDK native libraries and enabling warnings as errors on most of
them. However, some libraries, typically awt related, proved to be harder.
One of the reasons was that some toolchains had different warning labels
specified for C vs C++ and would not accept the other kind without
printing new warning. For libraries containing both C and C++ source
files, this prevented us from disabling all warnings. This limitation in
SetupNativeCompilation has since been fixed so I could use that
functionality to disable most remaining warnings.
Another reason is that some source files contain warnings that are
deemed severe enough by the compiler that they can't be individually
ignored. Since the team owning that source seem unwilling to fix these
warnings anyway, I have simply disabled all warnings on those specific
files, for the specific toolchains where it applies. I have tried to add
descriptive comments explaining each such occurrence.
I have tested this by building a full hotspot job and an openjdk only
job in JPRT. It is possible that people building with other toolchain
versions than Oracle will start seeing new build failures because of
more warnings in these libraries. In that case, either
--disable-warnings-as-errors, submit fixes for the warnings, or add more
warning ignores.
Bug: https://bugs.openjdk.java.net/browse/JDK-8148629
Webrev: http://cr.openjdk.java.net/~erikj/8148629/webrev.jdk.01/
/Erik
More information about the build-dev
mailing list