Merge pull request #1317 from vardbabayan/vb/1003-add-maybeunset-fromoption

feat: add MaybeUnset::from_option() to reduce boilerplate

Author: Lorak-mmk

Diff for: scylla-cql/src/serialize/value_tests.rs

@@ -2349,6 +2349,24 @@ fn unset_value() {
         do_serialize(set_i32, &ColumnType::Native(NativeType::Int)),
         vec![0, 0, 0, 4, 0, 0, 0, 32]
     );
+
+    let unset_option_i32: Option<i32> = None;
+    assert_eq!(
+        do_serialize(
+            MaybeUnset::from_option(unset_option_i32),
+            &ColumnType::Native(NativeType::Int)
+        ),
+        &(-2_i32).to_be_bytes()[..]
+    );
+
+    let set_option_i32: Option<i32> = Some(44);
+    assert_eq!(
+        do_serialize(
+            MaybeUnset::from_option(set_option_i32),
+            &ColumnType::Native(NativeType::Int)
+        ),
+        vec![0, 0, 0, 4, 0, 0, 0, 44]
+    );
 }
 
 #[test]

Diff for: scylla-cql/src/value.rs

@@ -33,6 +33,16 @@ pub enum MaybeUnset<V> {
     Set(V),
 }
 
+impl<V> MaybeUnset<V> {
+    #[inline]
+    pub fn from_option(opt: Option<V>) -> Self {
+        match opt {
+            Some(v) => Self::Set(v),
+            None => Self::Unset,
+        }
+    }
+}
+
 /// Represents timeuuid (uuid V1) value
 ///
 /// This type has custom comparison logic which follows Scylla/Cassandra semantics.