Review Request 128369: Handle multiple gzip streams in a single file
Martin Tobias Holmedahl Sandsmark
martin.sandsmark at kde.org
Tue Jul 5 11:25:48 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128369/
-----------------------------------------------------------
(Updated July 5, 2016, 11:25 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks and Sune Vuorela.
Changes
-------
Submitted with commit 853496897f5216909337ed8711e1b8391ae1b6a3 by Martin T. H. Sandsmark to branch master.
Repository: karchive
Description
-------
Handle multiple gzip streams
If kgzipfilter notices that zlib didn't read all the data, it tries to
re-init the stream and read the rest of the buffer. This is tested by
the unit test from Sune. The case where there's more than two streams
available in the current buffer is tested in a unit test added
separately.
If the split between the streams falls right between two buffers, we
need KCompressionDevice to notice that there's data left and try to
continue decompressing. This is easy to test by setting BUFFER_SIZE to
the size of the first stream in the unit test data (28 bytes).
Diffs
-----
autotests/kfiltertest.cpp 4408dbe
src/kcompressiondevice.cpp d6ecebd
src/kgzipfilter.cpp 27a6d99
Diff: https://git.reviewboard.kde.org/r/128369/diff/
Testing
-------
The unit test now passes.
Thanks,
Martin Tobias Holmedahl Sandsmark
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160705/932b6340/attachment.html>
More information about the Kde-frameworks-devel
mailing list