Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-05 Thread Martin Tobias Holmedahl Sandsmark
--- 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 ma

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-05 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128369/#review97114 --- Ship it! Ship It! - David Faure On July 4, 2016, 10:33

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-05 Thread David Faure
> On July 4, 2016, 9:39 p.m., David Faure wrote: > > src/kgzipfilter.cpp, line 331 > > > > > > This is where you could add something like > > if (avail_in < 10) { > > return Ok; // come back

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-04 Thread Martin Tobias Holmedahl Sandsmark
--- 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

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-04 Thread Martin Tobias Holmedahl Sandsmark
--- 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

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-04 Thread Martin Tobias Holmedahl Sandsmark
> On July 4, 2016, 9:39 p.m., David Faure wrote: > > src/kgzipfilter.cpp, line 331 > > > > > > This is where you could add something like > > if (avail_in < 10) { > > return Ok; // come back

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-04 Thread Martin Tobias Holmedahl Sandsmark
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128369/ --- (Updated July 4, 2016, 9:57 p.m.) Review request for KDE Frameworks and S

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-04 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128369/#review97103 --- Nice ! src/kgzipfilter.cpp (line 331)

Review Request 128369: Handle multiple gzip streams in a single file

2016-07-04 Thread Martin Tobias Holmedahl Sandsmark
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128369/ --- Review request for KDE Frameworks and Sune Vuorela. Repository: karchive