D21295: SyntaxHighlighter: Fix foreground color for text without special highlighting
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...
More information about the KWrite-Devel