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.

Sign up for GitHub

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

Jump to bottom

Use "trailing requires-clause" where we mean it #1672

Closed
hubert-reinterpretcast opened this issue Jul 18, 2017 · 1 comment
Closed

Use "trailing requires-clause" where we mean it #1672

hubert-reinterpretcast opened this issue Jul 18, 2017 · 1 comment
Assignees
@jensmaurer
Milestone
C++20 Concepts

Comments

@hubert-reinterpretcast
Copy link
Contributor

The instance of requires-clause in P0734R0 subclause 1.0.1 refers to the trailing requires-clause. Similarly for at least some other instances of requires-clause. For such cases, we should use trailing requires-clause instead.
@jensmaurer jensmaurer added this to the C++20 Concepts milestone Jul 18, 2017
@jensmaurer
Copy link
Member

Agreed.

jensmaurer added a commit to jensmaurer/draft that referenced this issue Nov 1, 2017
@jensmaurer
Use 'trailing requires-clause'
feb018b 
where the requires-clause in a template-head is not meant.

Fixes cplusplus#1672.
@jensmaurer jensmaurer mentioned this issue Nov 1, 2017
Merged
@jensmaurer jensmaurer self-assigned this Nov 1, 2017
zygoloid pushed a commit that referenced this issue Nov 12, 2017
@jensmaurer @zygoloid
Use 'trailing requires-clause'
ff65cec 
where the requires-clause in a template-head is not meant.

Fixes #1672.
AaronBallman pushed a commit to AaronBallman/draft that referenced this issue Nov 15, 2017
@jensmaurer
Use 'trailing requires-clause'
761efc0 
where the requires-clause in a template-head is not meant.

Fixes cplusplus#1672.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jensmaurer jensmaurer

Labels
None yet
Projects
None yet
Milestone
C++20 Concepts
Development

No branches or pull requests
2 participants
@hubert-reinterpretcast @jensmaurer