Version: 2.4.1 (using KDE 3.4.2 Level "b" , SUSE 10.0) Compiler: Target: x86_64-suse-linux OS: Linux (x86_64) release 2.6.13-15.7-default Realy nice job you'v done. Quite minor bug. Just try C following texte: switch (*sPtr) { case 0: break; // débugging case '@': case ' ': case'-': case '\n': case '_': case '.': break; default : if (*sPtr & 0x80) // Caractère ascii étendus sont autorisés... ; } } and see the coloration of the case following "case ' ': " Greet to the Kate dev team.
This bug can be closed in my opinion because there is simply a missing space between the case keyword and '-'...
Unfortunately, "case'-':" is still valid C code.
Proposed patch: ==== c.xml <comment name="multiLine" start="/*" end="*/" /> </comments> - <keywords casesensitive="1" /> + <keywords casesensitive="1" additionalDeliminator="'"" /> </general> </language>
Patch looks correct, as the standard delimiters are: \t.():!+,-<=>%&*/;?[]^{|}~\\ and space char. " and ' are not included. It can be committed imo, I believe it doesn't break other places.
SVN commit 562193 by kling: Fixed highlighting of "case'-':", which is ugly but allowed. Thanks to Matthew Woehlke for the fix. BUG: 121293 M +1 -1 c.xml --- branches/KDE/3.5/kdelibs/kate/data/c.xml #562192:562193 @@ -153,6 +153,6 @@ <comment name="singleLine" start="//" /> <comment name="multiLine" start="/*" end="*/" /> </comments> - <keywords casesensitive="1" /> + <keywords casesensitive="1" additionalDeliminator="'"" /> </general> </language>