Clear clippy warnings

renrs-gui/src/lib.rs

@@ -163,10 +163,10 @@ impl App {
             }
 
             let Vector2f { x, y } = shape.position();
-            if x >= WIDTH - SIZE || x < 0.0 {
+            if !(0.0..WIDTH - SIZE).contains(&x) {
                 velocity.x *= -1.0;
             }
-            if y >= HEIGHT - SIZE || y < 0.0 {
+            if !(0.0..HEIGHT - SIZE).contains(&y) {
                 velocity.y *= -1.0;
             }
             shape.set_position(Vector2f::new(x + velocity.x, y + velocity.y));

renrs/src/lib.rs

@@ -18,17 +18,19 @@ impl State {
         }
     }
 
-    pub fn next(&mut self) -> Option<Event> {
+    fn next_command(&mut self) -> Option<CommandContext> {
+        self.script.borrow_mut().next(&self.script)
+    }
+}
+impl Iterator for State {
+    type Item = Event;
+    fn next(&mut self) -> Option<Self::Item> {
         while let Some(command) = self.next_command() {
             let event = command.execute();
-            if let Some(_) = event {
+            if event.is_some() {
                 return event;
             }
         }
         None
     }
-
-    fn next_command(&mut self) -> Option<CommandContext> {
-        self.script.borrow_mut().next(&self.script)
-    }
-}
+}

renrs/src/parser/block.rs

@@ -78,7 +78,7 @@ impl CommandContext {
 }
 
 impl CommandBlock {
-    pub fn next<'a>(&mut self, self_rc: &Rc<RefCell<Self>>) -> Option<CommandContext> {
+    pub fn next(&mut self, self_rc: &Rc<RefCell<Self>>) -> Option<CommandContext> {
         let mut next = match self.next {
             Some(next) => next,
             None => return None,
@@ -98,7 +98,7 @@ impl CommandBlock {
                     next += 1;
                     break;
                 }
-                BlockElement::Block(block) => match block.borrow_mut().next(&block) {
+                BlockElement::Block(block) => match block.borrow_mut().next(block) {
                     Some(context_command) => {
                         result = Some(context_command);
                         break;
@@ -133,7 +133,7 @@ impl CommandBlock {
 
     pub fn get_root(&self, self_rc: &Rc<RefCell<Self>>) -> Rc<RefCell<Self>> {
         if let Some(parent) = &self.parent {
-            parent.borrow().get_root(&parent)
+            parent.borrow().get_root(parent)
         } else {
             self_rc.clone()
         }

renrs/src/parser/event.rs

@@ -19,8 +19,8 @@ impl Event {
                 let variables = context.get_variables();
                 *name = name
                     .as_deref()
-                    .map(|name| interpolate_string(&name, &variables));
-                *text = interpolate_string(&text, &variables);
+                    .map(|name| interpolate_string(name, &variables));
+                *text = interpolate_string(text, &variables);
             }
         }
         self

renrs/src/parser/utils.rs

@@ -4,7 +4,7 @@ use super::{
 };
 
 pub fn parse_line(pair: Pair) -> Vec<Token> {
-    pair.into_inner().map(|pair| parse_token(pair)).collect()
+    pair.into_inner().map(parse_token).collect()
 }
 
 // Line description e.g. [String, Keyword, Array]
@@ -12,10 +12,10 @@ pub fn parse_line(pair: Pair) -> Vec<Token> {
 pub fn describe_line(line: &[Token]) -> String {
     let mut description = "[".to_owned();
     let mut iter = line.iter();
-    description.push_str(&format!("{}", iter.next().unwrap().print()));
+    description.push_str(iter.next().unwrap().print());
     for token in iter {
         description.push_str(&format!(", {}", token.print()));
     }
-    description.push_str("]");
+    description.push(']');
     description
 }