Review Request 128369: Handle multiple gzip streams in a single file
Martin Tobias Holmedahl Sandsmark
martin.sandsmark at kde.org
Mon Jul 4 22:33:53 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128369/
-----------------------------------------------------------
(Updated July 4, 2016, 10:33 p.m.)
Review request for KDE Frameworks and Sune Vuorela.
Repository: karchive
Description (updated)
-------
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/20160704/5059c6e1/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list