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

[dcl.decl] Change "templated function" to instead use "templated entity" #3538

Merged
merged 1 commit into from Mar 4, 2020

Conversation

sdkrystian
Copy link
Contributor

@sdkrystian sdkrystian commented Dec 11, 2019

The resolution for CA378 used the term "templated function", but it is unclear what exactly this means (the intent is known, so I believe this is sufficiently editorial). This should be changed to "templated entity that is a function or function template" as is used in [dcl.dcl].

I didn't change the comment in the example to use this wording since it seems to be a little "extra" for a comment, but that can be changed too if deemed sufficiently important.

source/declarations.tex Outdated Show resolved Hide resolved
@zygoloid
Copy link
Member

The intent when introducing the term "templated entity" was that we could then use it to talk about "templated functions", "templated variables", and so on. We should define those terms rather than removing uses of them.

@sdkrystian
Copy link
Contributor Author

Perhaps we add this below the definition of templated entity:

A templated function is a templated entity that is a function or a function template. A templated variable is a templated entity that is a variable or a variable template. A templated class is a templated entity that is a class or a class template.

@zygoloid
Copy link
Member

Perhaps we add this below the definition of templated entity:

Yes, perhaps. Another option to consider is to define only the adjective "templated", not the noun phrase "templated entity". ("An entity is templated if it is [...]")

@zygoloid zygoloid added the decision-required A decision of the editorial group (or the Project Editor) is required. label Jan 14, 2020
@jensmaurer
Copy link
Member

jensmaurer commented Feb 10, 2020

Editorial meeting: Go with "An entity is templated if it is..."

@sdkrystian
Copy link
Contributor Author

@jensmaurer is a template or is templated?

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Feb 10, 2020
@sdkrystian sdkrystian closed this Feb 10, 2020
@sdkrystian sdkrystian reopened this Feb 10, 2020
@sdkrystian
Copy link
Contributor Author

Whoops, looks like deleting commits closes the PR. Changes applied.

@zygoloid zygoloid merged commit a4589f2 into cplusplus:master Mar 4, 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 this pull request may close these issues.

None yet

4 participants