zlib 1.2.3 -> 1.2.5

[Alan Bateman]

On 28/11/2011 22:06, Xueming Shen wrote:
> Hi,
>
> Here is the webrev for upgrading the bundled zlib from 1.2.3 to 1.2.5 
> for JDK8
>
> http://cr.openjdk.java.net/~sherman/7110149/webrev
>
> The changes to the original zlib source code (other than the copyright 
> attachment)
> is at
>
> http://cr.openjdk.java.net/~sherman/7110149/webrev/src/share/native/java/util/zip/zlib-1.2.5/patches/ChangeLog_java.html 
>
>
> For you reference, the 1.2.3 upgrade webrev can be found at
>
> http://cr.openjdk.java.net/~sherman/zlib123/webrev
>
> Thanks,
> Sherman
Thanks for doing this, the upgrade looks fine to me (assuming that 
zlib1.2.3 is deleted).

One concern about the test is that it would appear to be long running. I 
grabbed it and tried it on my system with zlib1.2.3 and it took about 
25s, and just over 1m on a much older/slower machine. Maybe the 
innermost loop could be dialed down, say to 5. It might be best to drop 
the /timeout=1000 as the right way to increase the timeout on really 
slow machines is to run with -timeoutFactor:N.

Minor comment on the test is that new Random(new 
Date().getTime()).nextBytes(dataIn) is a bit odd. Also might be nicer if 
check didn't declare that its throws Throwable.

When we talked about the zlib version here (I think back in February) we 
talked about having the option to use the system's zip library where 
appropriate. Have you given this any thought recently?

-Alan.