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