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

[2018-11 CWG Motion 15] P1141R2 Yet another approach for constrained declarations #2407

Closed
jensmaurer opened this issue Nov 10, 2018 · 7 comments · Fixed by #2453
Closed
Assignees
Milestone

Comments

@jensmaurer
Copy link
Member

jensmaurer commented Nov 10, 2018

P1141R2

@jensmaurer jensmaurer added this to the post-2018-11 milestone Nov 10, 2018
@jensmaurer jensmaurer assigned jensmaurer and unassigned jensmaurer Nov 11, 2018
@burblebee burblebee self-assigned this Nov 12, 2018
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 12, 2018

Note the interaction with motion CWG-4 (#2397). We'll need to adjust the wording of that paper after we apply this one.

@burblebee
Copy link
Contributor

I'll create the branch for this motion based on the paper as is, then I/someone can add the required changes as a separate edit, OK? @tkoeppe Can you suggest the needed changes (or just apply them to the branch yourself after I commit)?

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 12, 2018

Right, the fixup should probably appear in a separate commit (but in the same branch).

The point of contention is the grammar production qualified-concept-name that we are deleting in this paper, but which is being used by the earlier paper. We need to rephrase the affected sentence.

@hubert-reinterpretcast
Copy link
Contributor

-> type-constraint

If the return-type-requirement is of the form -> type-constraint, then the type-constraint applies to decltype((E)). The immediately-declared constraint of decltype((E)) shall be satisfied. [Note: ...]

@burblebee
Copy link
Contributor

@hubert-reinterpretcast Your suggested change looks correct, but I'm not sure about the wording, in particular "the type-constraint applies to decltype((E))" - can we find better wording than "applies" here? Perhaps wording that is closer to the original text?

@hubert-reinterpretcast
Copy link
Contributor

In the language of P1141:

-> type-constraint

If the return-type-requirement is of the form -> type-constraint, then the contextually-determined type being constrained is decltype((E)). The immediately-declared constraint of decltype((E)) shall be satisfied. [Note: ...]

@burblebee
Copy link
Contributor

burblebee commented Nov 16, 2018

@hubert-reinterpretcast Much better, thank you!

The wording changes were added to motions-2018-11-cwg-4 since it had the wording which needed to be changed.

zygoloid added a commit that referenced this issue Nov 25, 2018
P1141R2 Yet another approach for constrained declarations

Fixes #2407
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 a pull request may close this issue.

4 participants