dfaure added inline comments. INLINE COMMENTS
> deletejob.cpp:406 > + QMetaObject::invokeMethod(worker(), "rmfile", > Qt::QueuedConnection, > + Q_ARG(const QUrl&, m_currentURL), > + Q_ARG(bool, isLink)); Since you're already using m_currentURL here, I'd say this is acceptable. REPOSITORY R241 KIO BRANCH arcpatch-D24962 REVISION DETAIL https://phabricator.kde.org/D24962 To: meven, dfaure, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
