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

[unord.hash] [format.formatter.spec] Rephrase "specialization of X<Y>" #6257

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Quuxplusone
Copy link
Contributor

Both instances here have the same form: The first paragraph is phrased correctly as "for any type Y for which the library provides a specialization of X," and talks about disabling; the second paragraph talks about enabling in terms of "a specialization of X," which is incorrect. Flip it around so we talk enabling first, and use parallel construction in the two paragraphs.

Question: Can we replace the words "explicit or partial specialization" with the one word "specialization"? Would that be wrong or confusing somehow?

Both instances here have the same form: The first paragraph is
phrased correctly as "for any type Y for which the library provides
a specialization of X," and talks about disabling; the second
paragraph talks about enabling in terms of "a specialization of X<Y>,"
which is incorrect. Flip it around so we talk enabling first, and
use parallel construction in the two paragraphs.
Quuxplusone added a commit to Quuxplusone/draft that referenced this pull request May 28, 2023
This and cplusplus#6257 are the only instance of this solecism.
@JohelEGP
Copy link
Contributor

Question: Can we replace the words "explicit or partial specialization" with the one word "specialization"? Would that be wrong or confusing somehow?

Yeah. "Specialization" doesn't mean explicit specialization or partial specialization. It's defined at https://eel.is/c++draft/temp.spec.general#def:specialization.

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

2 participants