D14274: Modelines: add byte-order-mark & small fixes

Christoph Cullmann noreply at phabricator.kde.org
Mon Aug 13 07:13:15 BST 2018


cullmann requested changes to this revision.
cullmann added a comment.
This revision now requires changes to proceed.


  <DetectSpaces /> is only a performance optimization, as it just skips over whitespaces before any of the costly rules are tried to match. I think that should just be added again.
  Otherwise, I have no issues with this changes.
  What was exactly the problem with the unit test? Having one would be nice.

REPOSITORY
  R216 Syntax Highlighting

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

To: nibags, turbov, dhaumann, cullmann, vkrause, #kate
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/20180813/aefb6a28/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list