| Summary: | Syntaxic coloration error in "switch" statement | ||
|---|---|---|---|
| Product: | [Applications] kate | Reporter: | patrick serru <patrick> |
| Component: | general | Assignee: | KWrite Developers <kwrite-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version First Reported In: | 2.4.1 | ||
| Target Milestone: | --- | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
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>
|
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.