RFR: JDK-8232770 Enable more warnings on solaris studio
Erik Joelsson
erik.joelsson at oracle.com
Tue Oct 22 16:13:37 UTC 2019
Looks good.
/Erik
On 2019-10-22 02:35, Magnus Ihse Bursie wrote:
> As a part of the ongoing process of cleaning up and improving our
> handling of warnings, the time has now come to the solstudio compiler.
> (I know this is not the most relevant of compilers, but I actually
> started out with the warning cleanup project on solstudio, since it's
> a simple, but limited, compiler in terms of options. So this is an old
> personal branch that I'm finally bringing in to mainline.)
>
> Just as we have done with gcc (-Wall -Wextra), we should increase the
> general warning levels on solstudio, and then -- if needed -- disable
> individual warnings. Warnings should be disabled on a per-library
> level if they indicate real issues that should be fixed, or globally
> if they are deemed not relevant for the project as a whole.
> (Unfortunately, for solstudio, some warnings that really should be
> fixed applied to just about every library, and for sanity's sake I had
> to disable them globally.)
>
> I also deemed a few warnings that had previously been individually
> disabled, to be dumb enough (or at least unsuitable to us) to warrant
> global disabling.
>
> I intend to open separate bugs on the owners of the native libraries
> where new warnings have been disabled, since I believe they point to
> real bugs, or at least sloppy programming practices, and should be
> addressed.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8232770
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8232770-enable-more-solstudio-warnings/webrev.01
>
> /Magnus
More information about the build-dev
mailing list