On Fri, Oct 21, 2022 at 04:52:34PM +0100, Gavin Smith wrote: > > > > It is probably the converse that would be best, that is add > > contain_simple_text to shortcaption and math... Actually, > > it would probably be clearer if all the BRACE_arguments commands > > that are not contain_plain_text had contain_simple_text explictely. > > And also probably the same for line commands. I am a bit cautious not > > to add too many flags, but it would seem nice to me if there were more > > contain_* and in_* as flags, as the static table in command_data.txt is > > much easier to read than the corresponding codes with all the > > conditionals, in my opinion. > > Ok that makes sense.
I added the flags for all the commands and simplified accordingly the conditionals. I also took that opportunity to move some commands from containing simple text to the more restrictive containing plain text. It is less of a change as it may seem, as there was already parsing errors for those commands with @-commands on their lines, like "bad argument to...", so what this change really does is adding somewhat redundant warnings to the errors, but I think that it is more consistent like that. -- Pat
