D10857: Change qSort to std::sort in simplifiedUrlList
David Faure
noreply at phabricator.kde.org
Mon Feb 26 18:48:04 UTC 2018
dfaure requested changes to this revision.
dfaure added a comment.
This revision now requires changes to proceed.
I'm not opposed to the idea, but measuring CPU usage is a rather misleading indicator. What if it takes 10 times longer, because it's progressing much more slowly? ;)
Please at least measure with QElapsedTimer around the sorting (not to be committed, just to gather numbers about the actual performance of this from a user's point of view).
I'm interested in the result ;)
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D10857
To: jtamate, #frameworks, dfaure
Cc: markg, apol, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180226/833e17f7/attachment.html>
More information about the Kde-frameworks-devel
mailing list