D11543: Optimize many syntax highlighting files and fix the '/' char of SQL
Nibaldo González
noreply at phabricator.kde.org
Wed Mar 21 21:48:15 UTC 2018
nibags added a comment.
Then, in such a case, it would be better to make only conservative changes such as:
- Change StringDetect to DetectChar or Detect2Chars.
- Change AnyChar for DetectChar.
- Change RegExpr for StringDetect, DetectChar or Detect2Chars.
It would be better to undo some changes from RegExpr to WordDetect, DetectSpaces or Fallthrough to avoid regressions in older versions of KTextEditor. Because I do not believe that the difference in performance is so great as to justify the changes and stop supporting old versions.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D11543
To: nibags, #framework_syntax_highlighting, dhaumann
Cc: dhaumann, #frameworks, michaelh, genethomas, ngraham, cullmann, vkrause
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180321/92daeaa4/attachment.html>
More information about the Kde-frameworks-devel
mailing list