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.");
 	}
     });
+    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 p = pattern.write().unwrap();
+        *p = res.unwrap();
+    }
+    let lock_c = Arc::clone(&lock);
     loop {
-        let lock_c = Arc::clone(&lock);
         thread::sleep(time::Duration::from_millis(250));
-        let pattern = format_multiline("unit");
+        let pattern_c = Arc::clone(&pattern);
-        let res: Result<Vec<Box<dyn Pattern>>, ParseError> = pattern.iter()
+        let v = pattern_c.read().expect("could not read pattern");
-            .map(|x: &Vec<String>| parse_line((*x).clone()))
+        let mut lights = lock_c.write().unwrap();
-            .collect();
+        (*v).iter().for_each(move |x: Box<dyn Pattern>| {
-        if res.is_ok() {
+            x.execute(&mut lights);
-            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);
-            })
-        }
     }
 }