----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120382/#review67504 -----------------------------------------------------------
Ship it! Thanks for taking care of the forgotten folder \o/ thumbnail/comiccreator.cpp <https://git.reviewboard.kde.org/r/120382/#comment47068> this isn't needed; QTemporaryDir automatically removes in the destructor thumbnail/comiccreator.cpp <https://git.reviewboard.kde.org/r/120382/#comment47069> Why do we run unrar manually when we're already linking against karchive elsewhere? (I understand this is existing code, I'm just wondering if you know) - David Edmundson On Sept. 26, 2014, 8:16 p.m., Maarten De Meyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120382/ > ----------------------------------------------------------- > > (Updated Sept. 26, 2014, 8:16 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kio-extras > > > Description > ------- > > Mostly standard porting stuff. > I'll change the qdebug calls to qCDebug later. I'm planning to create a > global thumbnail logging component. > > > Diffs > ----- > > CMakeLists.txt b582741 > thumbnail/CMakeLists.txt cd1263a > thumbnail/comiccreator.cpp 39df58a > > Diff: https://git.reviewboard.kde.org/r/120382/diff/ > > > Testing > ------- > > Works fine on my small collection (only cbr files) > > > Thanks, > > Maarten De Meyer > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
