D12233: Avoid manipulation of lists with quadratic complexity

Stefan BrĂ¼ns noreply at phabricator.kde.org
Wed May 30 15:31:30 UTC 2018


bruns added inline comments.

INLINE COMMENTS

> dhaumann wrote in pendingfilequeue.cpp:69
> Did you check? This is certainly correct for i++ if i is an int. But for iterators, which are classes, this is often not the case, but I may be wrong here. In any case, this is a minor nitpick you can simply ignore, so please go ahead :-)

Yes, objdump output is identical.
It is likely true for any decent compiler when the iterator implementation is fully contained in the headers. The compiler is aware the iterator copy is unused and can drop the respective code.

REPOSITORY
  R293 Baloo

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

To: bruns, #baloo, michaelh
Cc: dhaumann, ngraham, kde-frameworks-devel, jtamate, bruns, ashaposhnikov, michaelh, astippich, spoorun
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180530/cc6c513f/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list