----------------------------------------------------------- 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
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel