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.init.list] Missing cv before T CWG2830 #3492

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

Conversation

sdkrystian
Copy link
Contributor

@sdkrystian sdkrystian commented Nov 19, 2019

[dcl.init.list] p3 says:

List-initialization of an object or reference of type T is defined as follows:

A "cv" should be inserted before T, so that when comparing to another type like is done in [dcl.init.list] p3 sub 2, the type will be compared ignoring cv-quals.

For instance, if T is const A and the element was const A, U would be A and T would be const A, leading to the proper initialization not being performed.

@zygoloid zygoloid added the cwg Issue must be reviewed by CWG. label Nov 25, 2019
@zygoloid
Copy link
Member

This change looks correct but not editorial.

@sdkrystian
Copy link
Contributor Author

sdkrystian commented Nov 25, 2019

@zygoloid I feel like this is just an oversight in the wording, since the intent is known.

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 12, 2023

CWG meeting consensus: needs a core issue, even if it's obvious what to do here.

@tkoeppe tkoeppe added the not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking. label Nov 12, 2023
@jensmaurer
Copy link
Member

CWG2830

@jensmaurer jensmaurer changed the title [dcl.init.list] Missing cv before T [dcl.init.list] Missing cv before T CWG2830 Nov 23, 2023
@frederick-vs-ja
Copy link
Contributor

Superseded by #6906 (b37a8ce).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cwg Issue must be reviewed by CWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants