D8059: KDevelop: abstractfilemanagerimport benchmark

René J.V. Bertin noreply at phabricator.kde.org
Fri Nov 3 12:43:39 UTC 2017


rjvbb added a comment.


  OK, will do - but then what about https://phabricator.kde.org/D8579? Should I just include the changes to the benchmark tool in this patch set?
  
  FWIW, I wrote my previous message on this ticket before I realised that calling the project filter from the ProjectWatcher class is redundant in the current implementation, since ProjectWatcher::addDir() is apparently called only with valid directories. That just means the whole point I was making in the previous message is moot and the ProjectWatcher class can simply be introduced as a means to improve the dirwatching algorithm just like it was intended originally.

REPOSITORY
  R32 KDevelop

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

To: rjvbb, #kdevelop, mwolff
Cc: mwolff, kdevelop-devel, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171103/bdfd8c0e/attachment.html>


More information about the KDevelop-devel mailing list