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.

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

Replace "typedef name" with <grammar>typedef-name</> #4401

Closed
tkoeppe opened this issue Dec 2, 2020 · 6 comments · Fixed by #4417
Closed

Replace "typedef name" with <grammar>typedef-name</> #4401

tkoeppe opened this issue Dec 2, 2020 · 6 comments · Fixed by #4417
Assignees

Comments

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 2, 2020

On a few occasions, we say "typedef name", which is never defined. Reflector discussion suggests that this is potential holdover from C, which infers English names from grammar names, but we don't do this in C++.

Resolution: replace "typedef name" with typedef-name everywhere.

@jensmaurer
Copy link
Member

(This has lots of conflicts with Davis' paper, so this needs to wait a little.)

@jensmaurer
Copy link
Member

@tkoeppe , do you want to similarly replace in "typedef name for linkage purposes", which is a stand-alone defined term?

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Dec 14, 2020

Is "typedef name for linkage purposes" actually defined? I couldn't find the definition, just circular references. (It's also not indexed.)

It looks like "{general} name for linkage purposes" is the defined term, and then "typedef name" is just an application of that -- in which case it suffers form the same problem as reported here. (If instead the whole phrase were a defined term, I'd be happy to keep it and use it.)

@jensmaurer
Copy link
Member

dcl.typedef p4

If the typedef declaration defines an unnamed class or enumeration, the first typedef-name declared by the declaration to be that type is used to denote the type for linkage purposes only (6.6).

Is very close to a definition.

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Dec 14, 2020

Hm, that just sounds like a definition that fills in "typedef-name" into "{general} name for linkage purposes", so that would suggest that we should indeed say "typedef-name for linkage purposes" everywhere. Otherwise we should add something to that paragraph like, "which is called the typedef name for linkage purposes [index]" (but that seems uncalled for).

@jensmaurer
Copy link
Member

I've made a suggestion for a proper definition in the linked pull request.

@jensmaurer jensmaurer self-assigned this Dec 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants