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

Don't try to forbid overloaded & in Destructible #382

Closed
CaseyCarter opened this issue Mar 14, 2017 · 1 comment
Closed

Don't try to forbid overloaded & in Destructible #382

CaseyCarter opened this issue Mar 14, 2017 · 1 comment

Comments

@CaseyCarter
Copy link
Collaborator

CaseyCarter commented Mar 14, 2017

Per LWG Kona consensus.

Proposed Resolution

(Relative to P0547R0) Change [concepts.lib.object.destructible] as follows:

template <class T>
concept bool Destructible() {
+ return is_nothrow_destructible<T>::value; // see below
- return is_nothrow_destructible<T>::value && // see below
-   requires(T& t, const remove_reference_t<T>& ct) {
-     { &t } -> Same<remove_reference_t<T>*>&&; // not required to be equality preserving
-     { &ct } -> Same<const remove_reference_t<T>*>&&; // not required to be equality preserving
-   };
}

Strike [concepts.lib.object.destructible]/p2 ("The expression requirement &ct ...") and [concepts.lib.object.destructible]/p3 ("n a (possibly const) lvalue t of type...").

@ericniebler
Copy link
Owner

@CaseyCarter for your review.

@CaseyCarter CaseyCarter added ready and removed open labels Jul 11, 2017
CaseyCarter added a commit that referenced this issue Jul 18, 2017
CaseyCarter added a commit that referenced this issue Jul 18, 2017
CaseyCarter added a commit that referenced this issue Jul 18, 2017
CaseyCarter added a commit that referenced this issue Jul 18, 2017
CaseyCarter added a commit that referenced this issue Jul 18, 2017
CaseyCarter added a commit that referenced this issue Jul 18, 2017
CaseyCarter added a commit that referenced this issue Jul 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants