D24403: Small performance improvements suggested by clang-tidy

Albert Astals Cid noreply at phabricator.kde.org
Fri Oct 4 22:49:23 BST 2019


aacid requested review of this revision.
aacid added a comment.
This revision is now accepted and ready to land.


  Ok, i'll commit it then :)

INLINE COMMENTS

> cullmann wrote in modebase.cpp:333
> 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?

right is a local var ^_^ i got confused by a similar-ish patch in qtbase where the qregexp was passed in.

REPOSITORY
  R39 KTextEditor

BRANCH
  master

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

To: aacid, cullmann, dhaumann
Cc: dhaumann, cullmann, kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191004/126d9798/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list