Skip to content

[class.conv.ctor] Turn last paragraph into a note #6505

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

Merged
merged 2 commits into from
Dec 17, 2024

Conversation

Eisenwave
Copy link
Member

Paragraph 1 already states:

draft/source/classes.tex

Lines 2347 to 2353 in 1c22d62

\pnum
A constructor that is not explicit\iref{dcl.fct.spec}
specifies a conversion from
the types of its parameters (if any)
to the type of its class.
Such a constructor is called a
\defnadj{converting}{constructor}.

The reader has no reason to believe that a copy/move constructor is excluded from the rule, and it certainly doesn't need to be stated in normative wording. The whole paragraph can and should be a note.

@tkoeppe tkoeppe self-assigned this Nov 11, 2023
@jensmaurer
Copy link
Member

jensmaurer commented Nov 22, 2023

CWG 2023-11-11: Approved as-is.

@jensmaurer jensmaurer added the after-motions Pull request is to be applied after the pending edits from WG21 straw polls have been applied. label Nov 22, 2023
@tkoeppe
Copy link
Contributor

tkoeppe commented Sep 26, 2024

@jensmaurer Would you prefer to take it as is, or wait for the improvements Davis has suggested?

@tkoeppe
Copy link
Contributor

tkoeppe commented Oct 16, 2024

@jensmaurer Ping

@jensmaurer
Copy link
Member

We can go forward as-is, and address Davis' concerns at a later stage. The important part is that CWG approved changing normative text to a note; fixing the note is then purely editorial.

@tkoeppe tkoeppe merged commit 55a58f9 into cplusplus:main Dec 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
after-motions Pull request is to be applied after the pending edits from WG21 straw polls have been applied.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants