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

Reply via email to