Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upDebug impl for HashMap::keys should not require values to be Debug #41924
Comments
Mark-Simulacrum
added
the
T-libs
label
Jun 22, 2017
sfackler
added
the
E-easy
label
Jun 22, 2017
This comment has been minimized.
This comment has been minimized.
|
Yep, seems overly restrictive. |
This comment has been minimized.
This comment has been minimized.
|
While we're on it, |
razielgn
referenced this issue
Jun 23, 2017
Merged
Relaxed Debug constraints on {HashMap,BTreeMap}::{Keys,Values}. #42854
bors
pushed a commit
that referenced
this issue
Jun 24, 2017
bors
added a commit
that referenced
this issue
Jun 24, 2017
bors
closed this
in
#42854
Jun 24, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
jonhoo commentedMay 11, 2017
#39002 landed
Debugimplementations for a number of derived types in collections, but imposes overly strict requirements. For example, forstd::collections::hash_map::Keys:Given that only the keys are printed, we should only require
K: Debug, not alsoV: Debug(unless I'm mistaken?).