[COMPRESS-39] A few upstream (Ant) changes to the ZIP package

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.0
• Assignee: Torsten Curdt
• Reporter: Stefan Bodewig
• Created: 30/Jan/09 15:15
• Updated: 13/Apr/10 20:42
• Resolved: 04/Feb/09 08:51

Description

I've recently applied three patches to the Ant code base that may be of interest to commons-compress as well.

http://svn.apache.org/viewvc?rev=738844&view=rev
use more modern collections (since Ant trunk now officially is JDK 1.4+). This results in some minor performance improvements because of less synchronization and a theoretical bigger improvement for entries with many extra fields.

http://svn.apache.org/viewvc?rev=738853&view=rev
make sure the archive is closed in the presence of a RuntimeExcepetion which may happen when LFH parsing fails.
Fixes https://issues.apache.org/bugzilla/show_bug.cgi?id=46559

http://svn.apache.org/viewvc?rev=739300&view=rev
skip extra fields that don't follow the recommended HEADER-ID, LENGTH structure at the end of the extra field (this is what unzip does as well).
Fixes https://issues.apache.org/bugzilla/show_bug.cgi?id=42940

There are two more issues open in Ant's bugzilla I may be working on over the next few weeks (low prio for me):
https://issues.apache.org/bugzilla/show_bug.cgi?id=46637 - which may be impossible to do in a backwards compatible way (not a concern for a sandbox component, I guess) and the more important https://issues.apache.org/bugzilla/show_bug.cgi?id=45429 which would allow archives to contain files bigger than 2 GB and support UTF8 filenames in a more portable way.

Technically I am a commons committer, maybe we can collaborate on the required changes via the commons dev list (or Ant's).