-
Notifications
You must be signed in to change notification settings - Fork 1
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
cxx20: return-type-requirement is not a type-constraint #88
Labels
ready to tackle
This story was discussed and can be immidietly tackled
Milestone
Comments
marehr
added
the
ready to tackle
This story was discussed and can be immidietly tackled
label
May 18, 2020
marehr
added a commit
to marehr/seqan3
that referenced
this issue
May 31, 2020
This PR fixes seqan/product_backlog#88, please see further explanations in that issue. With regex: "\{[ ]*(.+?)\s*\}\s*->\s*(.+);" and replace "requires std::same_as<decltype($1), $2>;"
marehr
added a commit
to marehr/seqan3
that referenced
this issue
May 31, 2020
This PR fixes seqan/product_backlog#88, please see further explanations in that issue. With regex: "\{[ ]*(.+?)\s*\}\s*->\s*(.+);" and replace "requires std::same_as<decltype($1), $2>;"
marehr
added a commit
to marehr/seqan3
that referenced
this issue
Jun 8, 2020
This PR fixes seqan/product_backlog#88, please see further explanations in that issue. With regex: "\{[ ]*(.+?)\s*\}\s*->\s*(.+);" and replace "requires std::same_as<decltype($1), $2>;"
marehr
added a commit
to marehr/seqan3
that referenced
this issue
Jun 9, 2020
This PR fixes seqan/product_backlog#88, please see further explanations in that issue. With regex: "\{[ ]*(.+?)\s*\}\s*->\s*(.+);" and replace "requires std::same_as<decltype($1), $2>;"
marehr
added a commit
to marehr/seqan3
that referenced
this issue
Jun 12, 2020
This PR fixes seqan/product_backlog#88, please see further explanations in that issue. With regex: "\{[ ]*(.+?)\s*\}\s*->\s*(.+);" and replace "requires std::same_as<decltype($1), $2>;"
6 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The following expression
is invalid starting with C++20.
In C++20 you would use the following expression :
(The return type needs a concept requirement)
The problem is that gcc 7 and 8 does not support the C++20 syntax.
A compromise that works for every compiler version is
Acceptance Criteria
Tasks
Definition of Done
The text was updated successfully, but these errors were encountered: