Stefan Monnier wrote:

> Well, your patch distinguishes between tcl-keyword-list and
> tcl-builtin-list, so is `namespace' more like `append' or more like
> `if'? To me it feels more like an `if', in that it seems to be part
> of the syntax rather than one of a set of builtin functions.

It seems it can be used in a similar fashion to "proc", eg

namespace eval foo {
   ... do stuff ...
}

or like a simple command:

namespace import foo::*

so IMO it could be either a builtin or keyword. I slightly prefer the
former, but don't have an especially strong opinion.

> But I honestly have no idea what was your intention behind the
> separation keyword-vs-builtin.

No particularly deep intention (as you might guess!). Partly to keep
the new stuff separate, partly that font-lock-builtin-face is not
otherwise used in tcl.el. Just seemed to make sense to me to use
builtin for these new ones.

> I know, but I think the other one should be changed as well. Except
> that in neither case do I know whether my suggestion is really the
> right idea. Maybe it breaks some hilighting, so please check it.

OK.

>>>> +          '("\\\\$" 0 font-lock-warning-face) ; escaped EOL

With regards to this (separate from the "get the right regexp" issue),
it can either:

i)  not be highlighted
ii) be highlighted in some other face. keyword or builtin (?) seem to be
the only non-obviously-wrong faces left.

I have no strong preference as to what Emacs should do. Personally, I
like such things strongly highlighted.

AFAICS, the situation for this is exactly the same in sh-script and
tcl, so the solution ought to be the same.


_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

Reply via email to