Closed jjpe closed 2 years ago
The motivation for this change is that given Refinement<T, P>, this manual impl imposes no demands on the P predicate type argument. The derive does impose such a constraint, as can be seen here. That has proven both undesirable and unnecessary.
Refinement<T, P>
P
I was actually hoping you hadn't merged this yet - there's a similar commit in the pipeline for impl std::fmt::Debug for Refinement<T, P> :)
impl std::fmt::Debug for Refinement<T, P>
The motivation for this change is that given
Refinement<T, P>
, this manual impl imposes no demands on theP
predicate type argument. The derive does impose such a constraint, as can be seen here. That has proven both undesirable and unnecessary.