dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land.
Makes sense. INLINE COMMENTS > main.cpp:151 > + for (; it != fileList.end(); ++it) { > + if (it->path == path) { > + break; (pre-existing) std::find_if with a lambda would be shorter and (imho) more readable, but feel free to ignore this if you're not at ease with that > main.cpp:166 > + const QString dest = copyJob->srcUrl().toString(); > + qDebug() << "path" << path; > + qDebug() << "dest" << dest; clean up the debug output please REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D7841 To: marten, #frameworks, dfaure Cc: elvisangelaccio, dfaure