Change the Option types' is_some and is_none methods to take self by reference (#16)

Author: babichjacob

deranged/src/lib.rs

@@ -1001,14 +1001,14 @@ macro_rules! impl_ranged {
 
             /// Returns `true` if the value is the niche value.
             #[inline(always)]
-            pub const fn is_none(self) -> bool {
+            pub const fn is_none(&self) -> bool {
                 <$type<MIN, MAX> as $crate::traits::RangeIsValid>::ASSERT;
                 self.get().is_none()
             }
 
             /// Returns `true` if the value is not the niche value.
             #[inline(always)]
-            pub const fn is_some(self) -> bool {
+            pub const fn is_some(&self) -> bool {
                 <$type<MIN, MAX> as $crate::traits::RangeIsValid>::ASSERT;
                 self.get().is_some()
             }

deranged/src/tests.rs

@@ -147,6 +147,18 @@ macro_rules! tests {
             assert!($opt::<5, 10>::None.is_none());
         )*}
 
+        #[test]
+        fn is_some_by_ref() {$(
+            let value = $opt::<5, 10>::Some($t::<5, 10>::MAX);
+            assert!($opt::is_some(&value));
+        )*}
+
+        #[test]
+        fn is_none_by_ref() {$(
+            let value = $opt::<5, 10>::None;
+            assert!($opt::is_none(&value));
+        )*}
+
         #[test]
         fn default() {$(
             assert_eq!($opt::<5, 10>::default(), $opt::<5, 10>::None);