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