dhaumann requested changes to this revision. dhaumann added a comment. This revision now requires changes to proceed.
In general ok, but there are two issues to be fixed 1. Remove spaces around items as noted in the comment 2. Please extend / add a highlighting test case in autotest/input/ The goal is to have all highlighting files unit tested, since otherwise we can not maintain this over time. INLINE COMMENTS > rest.xml:21 > + <!-- http://docutils.sourceforge.net/docs/ref/rst/directives.html --> > + <item> attention </item> > + <item> caution </item> Please remove the space before and after attention, i.e. " attention " -> "attention". Syntax highlighting files with spaces raise warnings and soon will not be accepted anymore. The reason for this change was that when loading the xml files, we want to avoid to trim hundreds of thousands of QStrings. Same for all the office items. REVISION DETAIL https://phabricator.kde.org/D7245 To: turbov, #kate, #framework_syntax_hightlighting, dhaumann Cc: dhaumann, #frameworks
