D15217: Tune KeywordList lookup

Christoph Cullmann noreply at phabricator.kde.org
Sun Sep 2 12:34:42 BST 2018


cullmann updated this revision to Diff 40852.
cullmann added a comment.


  Avoid saving the implicit capture group 0 on each regex match.
  That avoids allocating the match string each time (and the non-empty list).
  One anyways shall only access capture group >= 1

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15217?vs=40850&id=40852

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

AFFECTED FILES
  data/schema/language.xsd
  src/lib/definition.cpp
  src/lib/definition_p.h
  src/lib/keywordlist.cpp
  src/lib/keywordlist_p.h
  src/lib/rule.cpp
  src/lib/rule_p.h

To: cullmann, vkrause, dhaumann
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180902/9dde888f/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list