commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From M N <mach...@hotmail.com>
Subject Re: [compress] Added in-memory support for zip and 7z
Date Thu, 20 Oct 2016 21:11:09 GMT
Hi Stefan,


I have created tests for SeekableInMemoryByteChannel and spot small error.

Attached is a patch with tests and proposed fix.


Regarding thread safety I think would be more clear to remove volatile and document a class
as not thread safe.

I think the need for thread safe SeekableInMemoryByteChannel will be a rare case.


Cheers,

Maciej

Mime
View raw message