D25117: file ioslave: stop copying as soon as the ioslave is killed

David Faure noreply at phabricator.kde.org
Mon Feb 3 08:20:28 GMT 2020


dfaure added a comment.


  I wouldn't worry about the performance impact of the added if() statement, we're just testing a bool testMode which is always false (so the call to contains() will never happen outside tests) and the branch predictor will find out as soon as the second time, that it's always false.

REPOSITORY
  R241 KIO

BRANCH
  arcpatch-D25117

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

To: meven, #frameworks, dfaure, ngraham, apol
Cc: apol, 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/20200203/2c4db331/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list