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

Hyphenate 'pointer-to-member' when it is an adjective. #1386

Closed
wants to merge 3 commits into from

Conversation

jensmaurer
Copy link
Member

Fixes #1369.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jan 18, 2017

Looks good, will leave to @zygoloid to merge.

@@ -524,21 +524,21 @@
object. The null pointer value is converted to the null pointer value of
the destination type.

\rSec1[conv.mem]{Pointer to member conversions}
\rSec1[conv.mem]{Pointer-to-member conversions}
Copy link
Member

Choose a reason for hiding this comment

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

I'm concerned that this change suggests a false parallel: previously, all the "X-to-Y conversion"s converted an X into a Y. A pointer to member conversion definitely does not convert a pointer into a member. I'm also still trying to decide if the rules for adjectives really apply here; "pointer to member type" seems more like "office chair store" (which would not be written as "office-chair store") than like, say, "office-themed store".

We have quite a few other cases (such as "pointer to function types") that follow a similar non-hyphenation pattern. These seem quite natural to me as-is.

Copy link
Member Author

Choose a reason for hiding this comment

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

We have 15 appearances where "pointer-to-member" is, in fact, hyphenated, so I presume you would accept a patch that removes the hyphens there?

Copy link
Member

@zygoloid zygoloid Feb 6, 2017

Choose a reason for hiding this comment

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

Let's discuss this at Kona and agree on a rule, then apply it consistently.

Copy link
Contributor

Choose a reason for hiding this comment

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

We decided that we can get over this. Fine as written.

@tkoeppe tkoeppe added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Nov 12, 2017
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 12, 2017

I stole this commit, rebased it, added the definition of "pointer-to-member type", and committed this as cbd25c4.

@tkoeppe tkoeppe closed this Nov 12, 2017
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Nov 15, 2017
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

3 participants