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.
