D24403: Small performance improvements suggested by clang-tidy
Christoph Cullmann
noreply at phabricator.kde.org
Fri Oct 4 19:48:59 BST 2019
cullmann accepted this revision.
cullmann added a comment.
I think this is ok to merge, I don't see the issue with the QRegExp. (I understand that it stores it result in mutable data in the object, but that patch didn't alter this, just avoids the copying of the regex before this happens, or do I misread the diff?)
INLINE COMMENTS
> aacid wrote in modebase.cpp:333
> This is wrong, QRegExp is broken and lastIndexIn modifies the object even if the function is marked as const
Hmm, given that is a local var on the stack, I don't see an issue with this.
This would only be problematic with some static var, or?
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D24403
To: aacid, cullmann
Cc: cullmann, kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191004/2cd3145f/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list