JDK 9 RFR of JDK-8032734: Add cast lint warning to build of jdk repository

Dave Pointon dpointo8 at linux.vnet.ibm.com
Mon Jan 27 08:19:42 UTC 2014


Hi Joe ,

On Fri, 2014-01-24 at 13:45 -0800, Joe Darcy wrote:
> Hello,
> 
> Work has started to clear the jdk repo of the "cast" category of lint 
> warnings (JDK-8032733, "Fix cast lint warnings in client libraries"). 
> Anticipating the time when those warnings are cleared, I'd like to now 
> get a code review of JDK-8032734 "Add cast lint warning to build of jdk 
> repository" which will enable that warning in the build:
> 
> diff -r 78d907730378 make/Setup.gmk
> --- a/make/Setup.gmk    Fri Jan 24 11:03:15 2014 -0800
> +++ b/make/Setup.gmk    Fri Jan 24 13:41:24 2014 -0800
> @@ -27,7 +27,7 @@
> 
>   # To build with all warnings enabled, do the following:
>   # make JAVAC_WARNINGS="-Xlint:all -Xmaxwarns 10000"
> -JAVAC_WARNINGS := 
> -Xlint:-unchecked,-deprecation,-overrides,auxiliaryclass,classfile,dep-ann,divzero,empty,overloads,static,try,varargs 
> -Werror
> +JAVAC_WARNINGS := 
> -Xlint:-unchecked,-deprecation,-overrides,auxiliaryclass,cast,classfile,dep-ann,divzero,empty,overloads,static,try,varargs 
> -Werror
> 
>   # Any java code executed during a JDK build to build other parts of 
> the JDK must be
>   # executed by the bootstrap JDK (probably with -Xbootclasspath/p: ) 
> and for this
> 
> Thanks,
> 
> -Joe
> 

Anything to reduce the plethora of compile-time warnings - looks good to
me, albeit as a non-reviewer.

Rgds,

-- 
Dave Pointon FIAP MBCS

Now I saw, tho' too late, the folly of beginning a work before we count
the cost and before we we judge rightly of our strength to go thro'
with it - Robinson Crusoe




More information about the build-dev mailing list