move parse out of loop

src/main.rs

@@ -12,6 +12,7 @@ pub const BRIGHTNESS: u8 = 150;
 fn main() {
     let p: RawColor = [0, 0, 0, 0];
     let lock = Arc::new(RwLock::new([p; LED_SIZE]));
+    let pattern: Arc<RwLock<Vec<Box<dyn Pattern>>>> = Arc::new(RwLock::new(Vec::new()));
     let read = Arc::clone(&lock);
     thread::spawn(move || {
         let mut controller: Controller = ControllerBuilder::new()
@@ -29,19 +30,22 @@ fn main() {
         	run_lights(&mut controller, &lights).expect("Error running lights controller.");
 	}
     });
-    loop {
-        let lock_c = Arc::clone(&lock);
-        thread::sleep(time::Duration::from_millis(250));
-        let pattern = format_multiline("unit");
-        let res: Result<Vec<Box<dyn Pattern>>, ParseError> = pattern.iter()
+    let mul_string = format_multiline("unit");
+    let res: Result<Vec<Box<dyn Pattern>>, ParseError> = mul_string.iter()
         .map(|x: &Vec<String>| parse_line((*x).clone()))
         .collect();
     if res.is_ok() {
-            let mut lights = lock_c.write().unwrap();
-            let v: Vec<Box<dyn Pattern>> = res.unwrap();
-            v.iter().for_each(move |x| {
-                x.execute(&mut lights);
-            })
+        let mut p = pattern.write().unwrap();
+        *p = res.unwrap();
     }
+    let lock_c = Arc::clone(&lock);
+    let mut lights = lock_c.write().unwrap();
+    loop {
+        thread::sleep(time::Duration::from_millis(250));
+        let pattern_c = Arc::clone(&pattern);
+        let v = pattern_c.read().expect("could not read pattern");
+        (*v).iter().for_each(move |x: &Box<dyn Pattern>| {
+            x.execute(&mut lights);
+        });
     }
 }