<table><tr><td style="">cullmann requested changes to this revision.<br />cullmann added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D14593">View Revision</a></tr></table><br /><div><div><p>Thanks for the contribution.<br />
I think the version=... should just be an integer, just increment it by one.<br />
Beside that, it would be very nice to have some test file in the autotests folder like for other languages for this with the new features you cover (at least some parts would already be nice).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R216 Syntax Highlighting</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14593">https://phabricator.kde.org/D14593</a></div></div><br /><div><strong>To: </strong>pmoura, dhaumann, cullmann, vkrause, Framework: Syntax Highlighting, Kate<br /><strong>Cc: </strong>kwrite-devel, kde-frameworks-devel, pmoura, michaelh, genethomas, kevinapavew, ngraham, bruns, demsking, cullmann, vkrause, sars, dhaumann<br /></div>