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

[std] Use grammar typedef-name instead of 'typedef name'. #4417

Merged
merged 2 commits into from Jul 22, 2021

Conversation

jensmaurer
Copy link
Member

@jensmaurer jensmaurer commented Dec 14, 2020

The former includes names introduced by alias-declarations,
the latter (arguably) does not.

Fixes #4401

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 15, 2020

I do like this, but I feel it's a bit large and novel to get into the present mailing. Let's aim for the next mailing and get some more review feedback on this one.

source/basic.tex Outdated
@@ -1106,7 +1106,7 @@
\begin{codeblock}
typedef unsigned char T;
template<class T
= T // lookup finds the typedef name of \tcode{unsigned char}
= T // lookup finds the \grammarterm{typedef-name} of \tcode{unsigned char}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"of" seems awkward here. Is there a term like "resolving to" that we use for this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed "of unsigned char", which doesn't seem to add much in this example

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still see the "of unsigned char" here.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be fixed now.

source/classes.tex Outdated Show resolved Hide resolved
source/classes.tex Outdated Show resolved Hide resolved
@tkoeppe
Copy link
Contributor

tkoeppe commented Jan 14, 2021

@zygoloid, @burblebee: it'd be nice if we could land this; would you all be able to take another careful look?

@jensmaurer jensmaurer added this to the C++23 milestone May 29, 2021
@tkoeppe
Copy link
Contributor

tkoeppe commented May 29, 2021

@zygoloid, @burblebee, could you kindly review these changes? I'd like to aim to merge these after the next meeting's motions, for the post-meeting working draft.

@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Jun 15, 2021
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Jun 15, 2021
@jensmaurer jensmaurer modified the milestones: C++23, post-2021-06 Jun 15, 2021
@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Jun 18, 2021
@jensmaurer jensmaurer removed this from the post-2021-06 milestone Jun 18, 2021
Copy link
Member

@zygoloid zygoloid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me with or without the removal of "of unsigned char".

source/basic.tex Outdated
@@ -1106,7 +1106,7 @@
\begin{codeblock}
typedef unsigned char T;
template<class T
= T // lookup finds the typedef name of \tcode{\keyword{unsigned} \keyword{char}}
= T // lookup finds the \grammarterm{typedef-name} of \tcode{\keyword{unsigned} \keyword{char}}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We discussed removing the "of unsigned char" here but it's still present. Is that intentional?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. Removed for good.

The former includes names introduced by alias-declarations,
the latter (arguably) does not.
@jensmaurer jensmaurer requested a review from zygoloid July 21, 2021 19:12
@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Jul 21, 2021
@tkoeppe tkoeppe merged commit 4ccf57f into cplusplus:main Jul 22, 2021
@jensmaurer jensmaurer deleted the c13 branch July 22, 2021 13:23
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 this pull request may close these issues.

Replace "typedef name" with <grammar>typedef-name</>
4 participants