Review Request 125974: Make KTar KCompressionDevice-friendly
Romário Rios
luizromario at gmail.com
Mon Nov 23 14:57:28 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125974/
-----------------------------------------------------------
(Updated Nov. 23, 2015, 2:57 p.m.)
Status
------
This change has been discarded.
Review request for KDE Frameworks and Aleix Pol Gonzalez.
Repository: karchive
Description
-------
Up until now, since at least 5.12, decompressing some data coming directly from a QIODevice by using KCompressionDevice because this is a sequential device, and KTar and KArchive used to use QIODevice::seek and pos and some places, which made the decompression fail. This patch makes KTar sequential-friendly by replacing the calls to seek and pos with read and a simple counter, respectively.
Diffs
-----
src/karchive.cpp 0ece37c
src/ktar.cpp 824395e
Diff: https://git.reviewboard.kde.org/r/125974/diff/
Testing
-------
Makes the tests from review #125941 pass
Thanks,
Romário Rios
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151123/6930140b/attachment.html>
More information about the Kde-frameworks-devel
mailing list