D24962: [DeleteJob] Use a separate worker thread to run actual IO operation

David Faure noreply at phabricator.kde.org
Sun Oct 27 22:00:30 GMT 2019


dfaure added a comment.


  Any reason why you didn't implement my suggestion of
  
     DeleteJobIOWorker *ioworker() {
         if (!m_ioworker) {
               ...
         }
         return m_ioworker;
     }
    [...] 
    QMetaObject::invokeMethod(ioworker(), "rmfile", [...]);
  
  ?
  A call to an initSomething() method can easily be forgotten, while an on-demand getter ensure that the worker is created when it's needed (for the first time).
  Sorry for the nitpicking :-)

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D24962

To: meven, dfaure
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191027/2fb0f00c/attachment.html>


More information about the Kde-frameworks-devel mailing list