Move some impls into a new module

Move `impl<T> Value for Option<T>` into its own file.

execution-plan-traits/src/containers.rs

@@ -0,0 +1,44 @@
+//! Impl Value for various container types, if the inner type implements Value.
+
+use crate::{MemoryError, Primitive, Value};
+const NONE: &str = "None";
+const SOME: &str = "Some";
+
+impl<T> Value for Option<T>
+where
+    T: Value,
+{
+    fn into_parts(self) -> Vec<Primitive> {
+        match self {
+            Some(v) => {
+                let mut parts = Vec::new();
+                parts.push(SOME.to_owned().into());
+                parts.extend(v.into_parts());
+                parts
+            }
+            None => vec![NONE.to_owned().into()],
+        }
+    }
+
+    fn from_parts<I>(values: &mut I) -> Result<Self, MemoryError>
+    where
+        I: Iterator<Item = Option<Primitive>>,
+    {
+        let variant: String = values
+            .next()
+            .flatten()
+            .ok_or(MemoryError::MemoryWrongSize)?
+            .try_into()?;
+        match variant.as_str() {
+            NONE => Ok(None),
+            SOME => {
+                let val = T::from_parts(values)?;
+                Ok(Some(val))
+            }
+            other => Err(MemoryError::InvalidEnumVariant {
+                expected_type: "option".to_owned(),
+                actual: other.to_owned(),
+            }),
+        }
+    }
+}

execution-plan-traits/src/lib.rs

@@ -5,6 +5,7 @@ pub use self::primitive::{NumericPrimitive, Primitive};
 
 #[macro_use]
 mod primitive;
+mod containers;
 
 /// Types that can be written to or read from KCEP program memory.
 /// If they require multiple memory addresses, they will be laid out
@@ -18,48 +19,6 @@ pub trait Value: Sized {
         I: Iterator<Item = Option<Primitive>>;
 }
 
-const NONE: &str = "None";
-const SOME: &str = "Some";
-
-impl<T> Value for Option<T>
-where
-    T: Value,
-{
-    fn into_parts(self) -> Vec<Primitive> {
-        match self {
-            Some(v) => {
-                let mut parts = Vec::new();
-                parts.push(SOME.to_owned().into());
-                parts.extend(v.into_parts());
-                parts
-            }
-            None => vec![NONE.to_owned().into()],
-        }
-    }
-
-    fn from_parts<I>(values: &mut I) -> Result<Self, MemoryError>
-    where
-        I: Iterator<Item = Option<Primitive>>,
-    {
-        let variant: String = values
-            .next()
-            .flatten()
-            .ok_or(MemoryError::MemoryWrongSize)?
-            .try_into()?;
-        match variant.as_str() {
-            NONE => Ok(None),
-            SOME => {
-                let val = T::from_parts(values)?;
-                Ok(Some(val))
-            }
-            other => Err(MemoryError::InvalidEnumVariant {
-                expected_type: "option".to_owned(),
-                actual: other.to_owned(),
-            }),
-        }
-    }
-}
-
 /// TODO
 #[derive(Debug, thiserror::Error, Default)]
 pub enum MemoryError {