Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Decide as what kind of entity to annotate "derived-declarator-type-list" #1310

Closed
Eelis opened this issue Jan 1, 2017 · 1 comment
Closed

Decide as what kind of entity to annotate "derived-declarator-type-list" #1310

Eelis opened this issue Jan 1, 2017 · 1 comment
Assignees
@jensmaurer

Comments

@Eelis
Copy link
Contributor

Eelis commented Jan 1, 2017

Places where it is a \grammarterm:

declarators.tex:954:``\grammarterm{derived-declarator-type-list}
declarators.tex:1258:``\grammarterm{derived-declarator-type-list}
declarators.tex:1265:``\grammarterm{derived-declarator-type-list}
declarators.tex:1296:``\grammarterm{derived-declarator-type-list} \tcode{T}'',
declarators.tex:1303:``\grammarterm{derived-declarator-type-list}

Places where it is a \nonterminal:

declarators.tex:535:is ``\nonterminal{derived-declarator-type-list}
declarators.tex:539:is ``\nonterminal{derived-declarator-type-list cv-qualifier-seq} pointer to
declarators.tex:650:is ``\nonterminal{derived-declarator-type-list}
declarators.tex:654:is ``\nonterminal{derived-declarator-type-list} reference to
declarators.tex:847:is ``\nonterminal{derived-declarator-type-list}
declarators.tex:851:is ``\nonterminal{derived-declarator-type-list cv-qualifier-seq} pointer to member of class
declarators.tex:987:is ``\nonterminal{derived-declarator-type-list} array of
declarators.tex:997:is ``\nonterminal{derived-declarator-type-list} array of unknown bound of
declarators.tex:1000:The type ``\nonterminal{derived-declarator-type-list} array of
declarators.tex:1004:``\nonterminal{derived-declarator-type-list} array of unknown bound of

Possibly it should be neither? :)
@jensmaurer
Copy link
Member

Oh, \nonterminal is actually misused all over the place. We should use \grammarterm or \cv or \cvqual or ... See #1312.

Similar to derived-declarator-type-list, we also have parameter-type-list as a term of art that just happens to be spelled with hyphens. However, in the case of the derived-declarator-type-list, this is used more like a \placeholder.

@tkoeppe: Ok to use \placeholder here for now?

jensmaurer added a commit to jensmaurer/draft that referenced this issue Jan 1, 2017
@jensmaurer
[dcl.meaning] Consistently use \placeholder{derived-declarator-type-l…
a1e6f10 
…ist}.

Fixes cplusplus#1310.
@jensmaurer jensmaurer mentioned this issue Jan 1, 2017
Merged
@jensmaurer jensmaurer self-assigned this Jan 1, 2017
Eelis pushed a commit to Eelis/draft that referenced this issue Jan 31, 2017
@jensmaurer @Eelis
[dcl.meaning] Consistently use \placeholder{derived-declarator-type-l…
38051a4 
…ist}.

Fixes cplusplus#1310.
Eelis pushed a commit to Eelis/draft that referenced this issue Feb 5, 2017
@jensmaurer @Eelis
[dcl.meaning] Consistently use \placeholder{derived-declarator-type-l…
a93300b 
…ist}.

Fixes cplusplus#1310.
zygoloid pushed a commit that referenced this issue Feb 5, 2017
@jensmaurer @zygoloid
[dcl.meaning] Consistently use \placeholder{derived-declarator-type-l…
334ce66 
…ist}. (#1315)

Fixes #1310.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jensmaurer jensmaurer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Eelis @jensmaurer