Implement Encode & Decode for Reverse<T> types

Author: mzachar

src/de/impls.rs

@@ -16,6 +16,7 @@ use core::{
     ops::{Bound, Range, RangeInclusive},
     time::Duration,
 };
+use std::cmp::Reverse;
 
 impl Decode for bool {
     fn decode<D: Decoder>(decoder: &mut D) -> Result<Self, DecodeError> {
@@ -405,6 +406,18 @@ impl<'de, T: BorrowDecode<'de>> BorrowDecode<'de> for Wrapping<T> {
     }
 }
 
+impl<T: Decode> Decode for Reverse<T> {
+    fn decode<D: Decoder>(decoder: &mut D) -> Result<Self, DecodeError> {
+        Ok(Reverse(T::decode(decoder)?))
+    }
+}
+
+impl<'de, T: BorrowDecode<'de>> BorrowDecode<'de> for Reverse<T> {
+    fn borrow_decode<D: BorrowDecoder<'de>>(decoder: &mut D) -> Result<Self, DecodeError> {
+        Ok(Reverse(T::borrow_decode(decoder)?))
+    }
+}
+
 impl Decode for char {
     fn decode<D: Decoder>(decoder: &mut D) -> Result<Self, DecodeError> {
         let mut array = [0u8; 4];

src/enc/impls.rs

@@ -13,6 +13,7 @@ use core::{
     ops::{Bound, Range, RangeInclusive},
     time::Duration,
 };
+use std::cmp::Reverse;
 
 impl Encode for () {
     fn encode<E: Encoder>(&self, _: &mut E) -> Result<(), EncodeError> {
@@ -280,6 +281,12 @@ impl<T: Encode> Encode for Wrapping<T> {
     }
 }
 
+impl<T: Encode> Encode for Reverse<T> {
+    fn encode<E: Encoder>(&self, encoder: &mut E) -> Result<(), EncodeError> {
+        self.0.encode(encoder)
+    }
+}
+
 impl Encode for char {
     fn encode<E: Encoder>(&self, encoder: &mut E) -> Result<(), EncodeError> {
         encode_utf8(encoder.writer(), *self)

tests/basic_types.rs

@@ -4,6 +4,7 @@ use bincode::error::DecodeError;
 use core::cell::{Cell, RefCell};
 use core::ops::Bound;
 use core::time::Duration;
+use std::cmp::Reverse;
 use std::num::*;
 use utils::{the_same, the_same_with_comparer};
 
@@ -128,6 +129,21 @@ fn test_numbers() {
     the_same(Wrapping(5i64));
     the_same(Wrapping(5i128));
     the_same(Wrapping(5isize));
+
+    // Reverse types
+    the_same(Reverse(5u8));
+    the_same(Reverse(5u16));
+    the_same(Reverse(5u32));
+    the_same(Reverse(5u64));
+    the_same(Reverse(5u128));
+    the_same(Reverse(5usize));
+
+    the_same(Reverse(5i8));
+    the_same(Reverse(5i16));
+    the_same(Reverse(5i32));
+    the_same(Reverse(5i64));
+    the_same(Reverse(5i128));
+    the_same(Reverse(5isize));
 }
 
 #[test]