[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-19 Thread rpreissel
Github user rpreissel commented on the pull request: https://github.com/apache/commons-compress/pull/11#issuecomment-197507339 This is a memory issue and from my point of view there is no simple way to test it with an unit test. I have tried to write an unit test and used

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-19 Thread rpreissel
GitHub user rpreissel opened a pull request: https://github.com/apache/commons-compress/pull/11 Bugfix for COMPRESS-343 The class ...sevenz.Coders.DeflateDecoder does not close (end()) the Deflater and Inflater. This can lead to native memory issues: see https

[GitHub] commons-compress pull request: Bugfix for COMPRESS-343

2016-03-18 Thread rpreissel
Github user rpreissel commented on the pull request: https://github.com/apache/commons-compress/pull/11#issuecomment-197554132 I added tests with PowerMock/Mockito. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If