Commons Compress
  1. Commons Compress
  2. COMPRESS-39

A few upstream (Ant) changes to the ZIP package


    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0
    • Component/s: None
    • Labels:


      I've recently applied three patches to the Ant code base that may be of interest to commons-compress as well.
      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.
      make sure the archive is closed in the presence of a RuntimeExcepetion which may happen when LFH parsing fails.
      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).

      There are two more issues open in Ant's bugzilla I may be working on over the next few weeks (low prio for me): - which may be impossible to do in a backwards compatible way (not a concern for a sandbox component, I guess) and the more important 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).


        Torsten Curdt made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Stefan Bodewig made changes -
        Fix Version/s 1.0 [ 12313768 ]
        Dennis Lundberg made changes -
        Project Commons Sandbox [ 12310491 ] Commons Compress [ 12310904 ]
        Key SANDBOX-276 COMPRESS-39
        Component/s Compress [ 12311183 ]
        Stefan Bodewig made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Torsten Curdt made changes -
        Field Original Value New Value
        Assignee Torsten Curdt [ tcurdt ]
        Stefan Bodewig created issue -


          • Assignee:
            Torsten Curdt
            Stefan Bodewig
          • Votes:
            0 Vote for this issue
            0 Start watching this issue


            • Created: