D21295: SyntaxHighlighter: Fix foreground color for text without special highlighting

Christoph Cullmann noreply at phabricator.kde.org
Sun May 19 20:05:19 BST 2019

cullmann added a comment.

  I think one reason for the early out was that actually even the "isXXX" checks are very expensive, at least if I remember correctly they did show up a lot in my profiling in the past.
  Perhaps one should profile this once more (and if still visible) at least skip all the isXXX checks for the isDefault... case.
  Otherwise I have no issues with this.

  R216 Syntax Highlighting


To: dhaumann, vkrause, cullmann, kossebau
Cc: kwrite-devel, kde-frameworks-devel, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190519/ad100f46/attachment.html>

More information about the KWrite-Devel mailing list