Bug 26910 - re-merging java.util.zip
Summary: re-merging java.util.zip
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: libgcj (show other bugs)
Version: 4.2.0
: P3 normal
Target Milestone: 4.2.0
Assignee: Tom Tromey
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-28 18:58 UTC by Tom Tromey
Modified: 2007-03-28 02:19 UTC (History)
2 users (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed: 2006-03-28 19:15:13

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Tromey 2006-03-28 18:58:13 UTC 
Currently our java/util/zip/InflaterInputStream differs
from Classpath's.  Putting ours into Classpath causes regressions there.
Putting Classpath's into libgcj causes regressions in libgcj.
Probably one or both Inflater implementations is incorrect in some way.
Comment 1 Tom Tromey 2006-03-29 16:44:15 UTC 
Note that we need more mauve tests in this area before fixing the problem.
The current tests don't catch the bug(s).
Comment 2 Tom Tromey 2006-07-12 17:03:13 UTC 
This is no longer a problem.
I think the PR 27271 fix cleaned up the lingering problem.
(It is hard to be sure since this report is so vague...)
Comment 3 Mark Wielaard 2006-07-29 22:55:58 UTC 
I don't think this is fixed yet. It seems libgcj and classpath still have different nflaterInputStream implementations. See also the following thread were they were supposed to be merged, but a regression was found and the merge was reverted:
http://developer.classpath.org/pipermail/classpath-patches/2006-March/000940.html
Comment 4 Tom Tromey 2007-03-28 02:19:09 UTC 
All the problems were ironed out and now this is fixed.
As of 4.2 we only differ in Deflater and Inflater, which is
tolerable.