commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stefan Bodewig (JIRA)" <>
Subject [jira] [Commented] (COMPRESS-327) Support in-memory processing for ZipFile
Date Sun, 13 Dec 2015 16:59:46 GMT


Stefan Bodewig commented on COMPRESS-327:


AFAICT there isn't anything requiring Java7 in your patch - I had assumed you'd be using SeekableByteChannel.
 Sorry for the noise.

As usual we'll likely quibble about a few names (getFilePosition in SeekableInputStream) and
I'm not sure whether it wouldn't be better to turn SeekableInputStream into an interface -
haven't checked whether ZipFile/SevenZFile need any methods of InputStream that are not part
of SeekableInputStream.  But this can be easily resolved on the list.

I'd be happy if you applied the patch.

> Support in-memory processing for ZipFile
> ----------------------------------------
>                 Key: COMPRESS-327
>                 URL:
>             Project: Commons Compress
>          Issue Type: New Feature
>            Reporter: Brett Kail
>            Priority: Minor
>         Attachments: 0001-Add-a-SeekableInputStream-and-some-subclasses-that-Z.patch
> ZipFile (and SevenZFile) currently require a File argument, but it would be nice to support
in-memory byte buffers rather than requiring temp files.  Perhaps create a new SeekableInputStream
class (or SeekableDataInput interface) and add corresponding constructors.
> For convenience, perhaps also add a utility class that wraps a ByteBuffer and/or byte[]
and implements the new interface.
> (The sevenz package appears to have a similar limitation, so it might make sense to add
the support there at the same time, but I personally don't have a need for that.)

This message was sent by Atlassian JIRA

View raw message