[Lazarus] Syntax highlighting anomalies

[Martin]

  Thanks for the patch. I gonna look at it.

the changes to "message" are ok as they are.

The rest should be optional.
Instead of checkbocxes to switch them on/off, I probably just make them 
a new color setting. so you can set the attributes for break,exit.... 
under somethink like "Pascal Commands" (better idea welcome)
Same for the string types, Yes it seems a waste to have a new 
color-field just for those three, but I guess some people will like the 
old way (I do)

Martin


On 09/10/2010 12:26, Bernd Kreuss wrote:
> On 09.10.2010 12:43, Bernd Kreuss wrote:
>
>> The changes to the highlighter are just handful of lines, I quickly
>> hacked something together yesterday evening to see how this mode would
>> look like and it seems to work and I very much like how my programs
>> appear in this mode.
> Attached is a diff of my changes:
>
> * do not highlight String
> * do not highlight AnsiString
> * do not highlight WideString
> * highlight break
> * highlight continue
> * highlight exit
> * highlight message only in a procedure defnition (partial fix to the
> bug mentioned earlier, it does not eliminate all cases, judging from the
> comments in the source a complete fix would need an improved parser in
> the highlighter, but at least it now gets rid of highlighted "message"
> anywhere *outside* the line where a procedure is defined.
>
> the above patch is what I now use here locally and I like it much better
> than the original one, but I don't think these radical changes would be
> easily accepted by the core developers. It tries to highlight the ideas
> and intentions in the programmer's mind and not so much the strict and
> sometimes odd language definition.
>
> Bernd
>
>
> --
> _______________________________________________
> Lazarus mailing list
> Lazarus at lists.lazarus.freepascal.org
> http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20101009/5b561cb6/attachment-0004.html>