arc

src/main.rs

@@ -12,8 +12,6 @@ pub const BRIGHTNESS: u8 = 150;
 fn main() {
     let p: RawColor = [0, 0, 0, 0];
     let lock = Arc::new(RwLock::new([p; LED_SIZE]));
-    let lock_c = Arc::clone(&lock);
-
     thread::spawn(move || {
         let mut controller: Controller = ControllerBuilder::new()
     .channel(0, ChannelBuilder::new()
@@ -26,23 +24,24 @@ fn main() {
     .build()
     .expect("Could not construct LED Controller.");
 	loop {
+            let lock_c = Arc::clone(&lock);
         	let lights = lock_c.read().expect("Could not read array lock.");
         	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()
             .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();
-            for x in 0..v.len() {
-                let mut lights = lock.write().unwrap();
-                v[x].execute(&mut lights);
-            }
+            v.iter.for_each(move |x| {
+                x.execute(&mut lights);
+            })
         }
     }
 }