Disable overrides during jdk build

[Maurizio Cimadamore]

I think it makes sense, esp. if the messages appear to be redundant. The 
compiler logic is very strict and there are cases where it comes down to 
guessing user intent and compilers are notoriously bad at doing that. In 
the long term, I'd like to see @SuppressWarnings("overrides") applied in 
those cases where the impl knows what it's doing.

Maurizio

On 13/05/13 14:53, Chris Hegarty wrote:
> Please hold your fire! This is not a suggestion to about general 
> handling of warnings during the build, just a specific gripe I have 
> when trying to find genuine build failures among the noise.
>
> Would there be any objection to adding '-overrides' to the jdk build?
>
> This lint warning was added after the new build was introduced. I 
> suspect it would have been suppressed originally if it was supported 
> at the time.
>
> diff --git a/makefiles/Setup.gmk b/makefiles/Setup.gmk
> --- a/makefiles/Setup.gmk
> +++ b/makefiles/Setup.gmk
> @@ -23,7 +23,7 @@
>  # questions.
>  #
>
> -DISABLE_WARNINGS:=-Xlint:all,-deprecation,-unchecked,-rawtypes,-cast,-serial,-dep-ann,-static,-fallthrough,-try,-varargs,-empty,-finally 
>
> +DISABLE_WARNINGS:=-Xlint:all,-deprecation,-unchecked,-rawtypes,-cast,-serial,-dep-ann,-static,-fallthrough,-try,-varargs,-empty,-finally,-overrides 
>
>
>  # The generate old bytecode javac setup uses the new compiler to 
> compile for the
>  # boot jdk to generate tools that need to be run with the boot jdk.
>
> -Chris.
>
> P.S. how to handle warnings generally will have to be addressed at 
> some point, but I am not making any proposal at this time.