[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: leading and trailing spaces in brace commands
From: |
Gavin Smith |
Subject: |
Re: leading and trailing spaces in brace commands |
Date: |
Tue, 27 Sep 2022 20:10:57 +0100 |
On Tue, Sep 27, 2022 at 09:26:15AM +0200, Patrice Dumas wrote:
> > I'm not sure if it would be correct to ignore spaces for other commands:
> > @math, @value, @verb... Almost every command could be different. It
> > seems quite simple to say that these spaces are ignored for BRACE_arguments
> > commands but not for others, although needless to say I haven't fully
> > explored the consequences of this idea by trying to implement it.
>
> That's exactly what the implementation does currently, but my point is
> that some of these cocmmands should not be in BRACE_arguments but in
> another class, either one already existing, or a new one.
OK, some commands could move to BRACE_arguments if we want to ignore spaces
in their arguments.
@math isn't currently in BRACE_arguments and as spaces don't need to be
ignored for @math (I haven't checked whether they are or not), it's fine
to leave it out.
I see you have made some changes to the command types. @value and @verb are
now BRACE_other (along with @w). I guess @value is special because it
is expanded immediately, and @verb and @w also are handled uniquely.
I looked through command_data.txt and your list of commands
where spaces around arguments can be kept - @dmn, @key, @indicateurl,
@titlefont - appears sensible to me. I don't know how easy it would be
to change any of these. Would you be moving them to BRACE_style_other?