D15217: Tune KeywordList lookup

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


cullmann created this revision.
cullmann added reviewers: vkrause, dhaumann.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
cullmann requested review of this revision.

REVISION SUMMARY
  No longer any allocations, we use sorted vectors for lookup.
  Init sorted vectors lazy, but no longer as lazy that mutable members are needed.

TEST PLAN
  make && make test works
  perf looks nicer afterwards

REPOSITORY
  R216 Syntax Highlighting

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

AFFECTED FILES
  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/kwrite-devel/attachments/20180902/a4aced0b/attachment.html>


More information about the KWrite-Devel mailing list