lint

1 files changed, 16 insertions, 17 deletions

diff --git a/src/sim/materials/fire.rs b/src/sim/materials/fire.rs
index 3c686fe..97027d5 100644
--- a/src/sim/materials/fire.rs
+++ b/src/sim/materials/fire.rs
@@ -12,7 +12,7 @@ impl FireCellView for Cell {
     fn get_ticks_lived(self) -> u16 {
         self.data
     }
-    fn set_ticks_lived(&mut self, ticks: u16) -> () {
+    fn set_ticks_lived(&mut self, ticks: u16) {
         self.data = ticks;
     }
     // this could be a property of the material def, I think it's better here for now
@@ -52,22 +52,21 @@ pub fn sim_update(ctx: &mut UpdateCtx) {
     }
 
     // 8 times in our lifespan, emit smoke
-    if ticks_lived % 30 == 0 {
-        if let Some(target) = ctx.get_cell(0, -1)
-            && target.material == MaterialId::Void
-        {
-            for (dx, dy) in [
-                (0, -1),
-                (1 - ctx.seqno_parity as i32 * 2, 0),
-                (-1 + ctx.seqno_parity as i32 * 2, 0),
-                (0, 1),
-            ] {
-                if let Some(target) = ctx.get_cell(dx, dy)
-                    && target.material == MaterialId::Void
-                {
-                    ctx.set_cell(dx, dy, Cell::from_material(MaterialId::Smoke));
-                    break;
-                }
+    if ticks_lived.is_multiple_of(30)
+        && let Some(target) = ctx.get_cell(0, -1)
+        && target.material == MaterialId::Void
+    {
+        for (dx, dy) in [
+            (0, -1),
+            (1 - ctx.seqno_parity as i32 * 2, 0),
+            (-1 + ctx.seqno_parity as i32 * 2, 0),
+            (0, 1),
+        ] {
+            if let Some(target) = ctx.get_cell(dx, dy)
+                && target.material == MaterialId::Void
+            {
+                ctx.set_cell(dx, dy, Cell::from_material(MaterialId::Smoke));
+                break;
             }
         }
     }