Skip to content

Commit 88898e6

Browse files
committed
Auto merge of #28892 - cristicbz:default-wrapping, r=alexcrichton
It's not very common to store `Wrapping` values, but I kept wrapping and unwrapping a hash value when I taking it out of a struct to do operations on it. I couldn't store the hash as `Wrapping<u64>` because I wanted to be able to `#[derive(Default)]` for the struct. At any rate, it feels to me that `Wrapping<T>` should implement pretty much everything `T` does. I left out `#[derive(Hash)]` since I'd be hard pressed to find a use case and wanted to avoid the extra generated code, but maybe I should add that too?
2 parents 2d405af + e802180 commit 88898e6

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/libcore/num/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ use slice::SliceExt;
3939
/// all standard arithmetic operations on the underlying value are
4040
/// intended to have wrapping semantics.
4141
#[stable(feature = "rust1", since = "1.0.0")]
42-
#[derive(PartialEq, Eq, PartialOrd, Ord, Clone, Copy, Debug)]
42+
#[derive(PartialEq, Eq, PartialOrd, Ord, Clone, Copy, Debug, Default)]
4343
pub struct Wrapping<T>(#[stable(feature = "rust1", since = "1.0.0")] pub T);
4444

4545
pub mod wrapping;

0 commit comments

Comments
 (0)