[7u10] Request for approval for CR 7188852 (original) (raw)

[7u10] Request for approval for CR 7188852 - Move implementation of De/Inflater.getBytesRead/Writtten() to java from native

Seán Coffey sean.coffey at oracle.com
Wed Sep 19 07:19:29 PDT 2012

Andrew,

would it be possible to combine your 7110151 & 7188852 into one approval request (2 changesets) so we get an all or nothing outcome here ? I see you're pushing for 7u10 and if you've strong justification, I can help push for 7u10 integration approval. You'll need to get review from the original reviewers also.

regards, Sean.

On 19/09/12 01:38, Andrew Hughes wrote:

The MacOS X changes have added an option to jdk7u to use the system zlib library via setting SYSTEMZLIB=true. However, at present, this is broken on 64-bit GNU/Linux platforms, such that setting SYSTEMZLIB=true causes a build failure.

I've already pushed a fix for this (7110151) to jdk8, which required a pre-requisite fix (7188852) to remove local changes to the zlib library. This is the first of two requests to get these fixes into 7u. 7188852, the subject of this request, moves the tracking of bytes read/written into Java code from native code. Standard system zlib tracks these values, but has an issue with possible overflow when the size of the zip goes over 4GB, such that the developers recommend that the application provides its own tracking: http://zlib.net/zlibfaq.html#faq32 At present, 7u has a fix applied to the in-tree zlib code to change the type of these fields. 7188852 removes this fix and instead implements tracking in the Java code. Bug report: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=7188852 Public review: http://mail.openjdk.java.net/pipermail/core-libs-dev/2012-August/011011.html Webrev: http://cr.openjdk.java.net/~andrew/7188852/webrev.01/ The webrev is nearly identical to the original jdk8 changeset: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/1468b0af0d06 The only necessary changes were to fix the paths to the zlib code (7u has 1.2.3, 8 has 1.2.5) and to apply the changes to Changelogjava & inflate.c manually, as the context differed slightly. The 7u webrev builds and the included test case passes. Ok for 7u10? Thanks,

More information about the jdk7u-dev mailing list