D8043: KDirWatch : make methods virtual

David Faure noreply at phabricator.kde.org
Thu Sep 28 22:45:52 UTC 2017


dfaure requested changes to this revision.
dfaure added a comment.


  In addition to being BIC, this is not the way to add thread safety. Either the class should be threadsafe (and then it should use mutexes internally) or it doesn't need to be threadsafe (and that can just be documented).
  Why not create one KDirWatch per thread?

REPOSITORY
  R244 KCoreAddons

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

To: rjvbb, #frameworks, mwolff, dfaure
Cc: dfaure, mwolff, kde-frameworks-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170928/bea39007/attachment.html>


More information about the Kde-frameworks-devel mailing list