RFR 9: 8074818: Resolve disabled warnings for libjava
Alan Bateman
Alan.Bateman at oracle.com
Fri May 22 06:55:08 UTC 2015
On 22/05/2015 01:42, Roger Riggs wrote:
> Oops, got the wrong host:
>
> Webrev:
> http://cr.openjdk.java.net/~rriggs/webrev-fix-all-warnings-8074818/
>
> Issues:
> 8074818: Resolve disabled warnings for libjava
> 8080007: Stop ignoring warnings for libjava
>
> Thanks, Roger
In JDK_GetVersionInfo0 then I wonder if we should change this assert to
be a fatal error on product builds. Periodically people set the build to
numbers > 255 and often only see issues when they use a fastdebug build.
Th rest looks okay to me. I don't particularly like the IOE_FORMAT in
ProcessImpl.c but I think other areas have done similar to deal with
this warning. ConcurrentPReader_md.c is being removed in another patch
under review at the moment so might be gone before you push.
-Alan.
More information about the build-dev
mailing list