[over.call.func, gram.key] Make colon in bnf non-italic. #4385
Conversation
Eelis
changed the title
There was a problem hiding this comment.
For "normal" bnf, we have \nontermdef for the initial thing.
"ncbnf" are the non-copied BNF parts (those that aren't copied to Annex A), but we should have a similar macro defined there, instead of hand-adding a colon or (gosh) hand-changing it to \textnormal.
|
We could make the colon an “active character.” This would allow you to just type an unadorned colon in the source (within the BNF environments) and LaTeX would interpret it as a macro that expands to |
|
@jensmaurer's suggestion sounds somewhat more structured -- what do you think? That way people don't have to remember to type colons at all. |
|
As long as |
source/overloading.tex
Outdated
| @@ -745,7 +745,7 @@ | |||
| parentheses, has one of the following forms: | |||
|
|
|||
| \begin{ncbnf} | |||
| postfix-expression:\br | |||
| postfix-expression\textnormal{:}\br | |||
There was a problem hiding this comment.
Does this line add any value? Can it be removed?
By moving \fmtnontermdef to macros.tex and renaming it to \ninontermdef, for "Non-Indexed \nontermdef", analogous to how ncbnf means "Non-Copied bnf".
|
I noticed there was already a non-indexed version of \nontermdef used for format nontermdefs: \fmtnontermdef. So I've changed the patch to:
|
| @@ -573,7 +573,8 @@ | |||
|
|
|||
| \newenvironment{bnfbase} | |||
| { | |||
| \newcommand{\nontermdef}[1]{{\BnfNontermshape##1\itcorr}\indexgrammar{\idxgram{##1}}\textnormal{:}} | |||
| \newcommand{\nontermdef}[1]{{\BnfNontermshape##1\itcorr}\indexgrammar{\idxgram{##1}}\textnormal{:}}% | |||
There was a problem hiding this comment.
Drop the trailing percent; it has no use inside macros.tex
| @@ -19238,7 +19238,6 @@ | |||
| % FIXME: For now, keep the format grammar productions out of the index, since | |||
| % they conflict with the main grammar. | |||
| % Consider renaming these en masse (to fmt-* ?) to avoid this problem. | |||
| \newcommand{\fmtnontermdef}[1]{{\BnfNontermshape#1\itcorr}\textnormal{:}} | |||
There was a problem hiding this comment.
I would prefer if you would leave the uses of \fmtnontermdef as-is and just changed its definition to be \ninontermdef.
(This keeps the semantic markup of the fmt-related grammar.)
Maybe we want to add an index of fmt-related grammar at some later time.
jensmaurer
added
the
changes requested
wg21bot
added
the
needs rebase
No description provided.