Rename marked_for_update -> layout_update_flags

Also remove the unnecessary flag vector reallocation

Author: llMBQll

omni-led-lib/src/script_handler/script_handler.rs

@@ -31,7 +31,7 @@ struct DeviceContext {
     name: String,
     layouts: Vec<Layout>,
     animation_groups: Vec<HashMap<usize, AnimationGroup>>,
-    marked_for_update: Vec<bool>,
+    layout_update_flags: Vec<bool>,
     time_remaining: Duration,
     last_priority: usize,
     state: State,
@@ -116,7 +116,7 @@ impl ScriptHandler {
         for device in &mut self.devices {
             for (index, layout) in device.layouts.iter().enumerate() {
                 if layout.run_on.contains(key) {
-                    device.marked_for_update[index] = true;
+                    device.layout_update_flags[index] = true;
                 }
             }
         }
@@ -133,7 +133,9 @@ impl ScriptHandler {
     fn reset(&mut self, device_name: &String) {
         match self.devices.iter_mut().find(|x| x.name == *device_name) {
             Some(ctx) => {
-                ctx.marked_for_update = vec![false; ctx.marked_for_update.len()];
+                for marked_for_update in &mut ctx.layout_update_flags {
+                    *marked_for_update = false;
+                }
                 ctx.time_remaining = Duration::ZERO;
                 ctx.last_priority = 0;
                 ctx.state = State::Finished;
@@ -144,7 +146,7 @@ impl ScriptHandler {
         }
     }
 
-    pub(self) fn register(
+    fn register(
         &mut self,
         lua: &Lua,
         device_name: String,
@@ -160,7 +162,7 @@ impl ScriptHandler {
             name: device_name,
             layouts,
             animation_groups: vec![HashMap::new(); layout_count],
-            marked_for_update: vec![false; layout_count],
+            layout_update_flags: vec![false; layout_count],
             time_remaining: Default::default(),
             last_priority: 0,
             state: State::Finished,
@@ -188,18 +190,15 @@ impl ScriptHandler {
         ctx.time_remaining = ctx.time_remaining.saturating_sub(time_passed);
         let has_time_remaining = !ctx.time_remaining.is_zero();
 
-        let mut marked_for_update = vec![false; ctx.marked_for_update.len()];
-        std::mem::swap(&mut ctx.marked_for_update, &mut marked_for_update);
-
         let mut to_update = None;
         let mut new_update = false;
-        for (priority, marked_for_update) in marked_for_update.into_iter().enumerate() {
+        for (priority, marked_for_update) in ctx.layout_update_flags.iter().enumerate() {
             // If a more important layout still has time remaining, don't bother checking further
             if has_time_remaining && ctx.last_priority < priority {
                 break;
             }
 
-            if marked_for_update && Self::test_predicate(&ctx.layouts[priority].predicate)? {
+            if *marked_for_update && Self::test_predicate(&ctx.layouts[priority].predicate)? {
                 to_update = Some(priority);
                 new_update = true;
                 break;
@@ -220,6 +219,10 @@ impl ScriptHandler {
             }
         }
 
+        for marked_for_update in &mut ctx.layout_update_flags {
+            *marked_for_update = false;
+        }
+
         let to_update = match to_update {
             Some(to_update) => to_update,
             None => return Ok(()),