Update of patch #4970 (project mc):
Status:None = Done
Assigned to:None = leonardjo
Open/Closed:Open = Closed
Follow-up Comment #6, patch #4970 (project mc):
Sorry Oswald, but i have to say: actually, the highlighting code sucks ;-)
Ok. if we change the rule to:
keyword whole \{A...z\}\[0...z\]: cyan
it highlights the case labels and ( ... ? a: b) if 'a' and the colon does not
separated.
I do not
Follow-up Comment #7, patch #4970 (project mc):
Gergely, any perceived flaws in the syntax highlighting code are no excuse
for creating a faulty patch that obviously doesn't work with the ruleset.
Your patch adds an earlier match overriding the matching of the default
keyword. This has nothing
Follow-up Comment #4, patch #4970 (project mc):
Ok. So, it should be fixed. Gergely ?
___
Reply to this item at:
http://savannah.gnu.org/patch/?func=detailitemitem_id=4970
___
Message sent
Follow-up Comment #5, patch #4970 (project mc):
Fixing this would require to distinguish between contexts or a negative match
as in !default which I'm not aware of (not saying it isn't implemented).
Even adding an extra colon to the default keyword doesn't fix the issue,
because the added white
Follow-up Comment #1, patch #4970 (project mc):
Any comments from the syntax gurus ?
___
Reply to this item at:
http://savannah.gnu.org/patch/?func=detailitemitem_id=4970
___
Message sent
Follow-up Comment #3, patch #4970 (project mc):
actually, this patch sucks - it breaks the highlighting of the default:
labels in switch statements.
___
Reply to this item at:
http://savannah.gnu.org/patch/?func=detailitemitem_id=4970
URL:
http://savannah.gnu.org/patch/?func=detailitemitem_id=4970
Summary: add label highlighting to c.syntax
Project: GNU Midnight Commander
Submitted by: szaszg
Submitted on: Sun 03/12/06 at 19:42
Category: Editor
