-
Notifications
You must be signed in to change notification settings - Fork 805
P3921R0 issue 1670 #8490
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
base: main
Are you sure you want to change the base?
P3921R0 issue 1670 #8490
Conversation
Fixes NB FI-4 (C++14 CD).
7955549 to
c40f633
Compare
|
Currently, there's one such use made invalid by CWG1670 in the library specification. Line 850 in 605dcaa
It seems OK to replace |
|
That's technically a normative change, and I would prefer to keep this PR minimal so we don't run into anything that prevents us from merging motions. The fix of [class.conv.fct] was clearly just a minor editorial adjustment, but the |
|
You know what, I'm just going to submit an LWG issue for this. |
|
We have now LWG 4468 for this. |
|
Thanks, @Dani-Hub! |
|
It seems to me that the entire paragraph "A conversion function template shall not have a deduced return type" should go away. Technically, that might (?) actually be a normative change in that the current rule makes the template |
I wouldn't be opposed, but that seems more like CWG issue material than a drive-by fix in this PR which applies the motions. |
Fixes #8456.
Fixes FI-4 (C++14 CD).
Note that this PR also fixes an example in [class.conv.fct] that becomes clearly incorrect after CWG1670 is applied.