Created on 2008-10-06.00:00:00 last changed 171 months ago
Proposed resolution:
Replace in [forwardlist.ops] before 11 and in [list.ops] before 15
requiresEqualityComparable<T>HasEqualTo<T, T> void remove(const T& value);
[ 2009-10 Santa Cruz: ]
NAD, solved by the removal of concepts.
[ 2009-10-10 Daniel adds: ]
Recommend NAD: The concept-free wording as of N2960 has no longer the over-specified requirement EqualityComparable for the remove function that uses ==. In fact, now the same test conditions exists as for the free algorithm remove ([alg.remove]). The error was introduced in the process of conceptifying.
[ 2009-07-21 Alisdair adds: ]
Current rationale and wording for this issue is built around concepts. I suggest the issue reverts to Open status. I believe there is enough of an issue to review after concepts are removed from the WP to re-examine the issue in Santa Cruz, rather than resolve as NAD Concepts.
[ Batavia (2009-05): ]
We agree with the proposed resolution, but would like additional input from concepts experts.
Move to Review.
The signatures of forwardlist::remove and list::remove defined in [forwardlist.ops] before 11 + [list.ops] before 15:
requires EqualityComparable<T> void remove(const T& value);
are asymmetric to their predicate variants (which only require Predicate, not EquivalenceRelation) and with the free algorithm remove (which only require HasEqualTo). Also, nothing in the pre-concept WP N2723 implies that EqualityComparable should be the intended requirement.
History | |||
---|---|---|---|
Date | User | Action | Args |
2010-10-21 18:28:33 | admin | set | messages: + msg4368 |
2010-10-21 18:28:33 | admin | set | messages: + msg4367 |
2010-10-21 18:28:33 | admin | set | messages: + msg4366 |
2010-10-21 18:28:33 | admin | set | messages: + msg4365 |
2010-10-21 18:28:33 | admin | set | messages: + msg4364 |
2008-10-06 00:00:00 | admin | create |