manual clippy and other cleanup

1 files changed, 15 insertions, 23 deletions

diff --git a/pixel-client/src/game.rs b/pixel-client/src/game.rs
index 32760364..0c5661c2 100644
--- a/pixel-client/src/game.rs
+++ b/pixel-client/src/game.rs
@@ -212,11 +212,9 @@ impl Game {
         });
 
         for player in self.players.values_mut() {
-            for item in &mut player.items {
-                if let Some(item) = item {
-                    item.parent_position = player.movement.position;
-                    item.tick(1., dt);
-                }
+            for item in player.items.iter_mut().flatten() {
+                item.parent_position = player.movement.position;
+                item.tick(1., dt);
             }
         }
         for tile in self.tiles.values_mut() {
@@ -360,10 +358,8 @@ impl Game {
                 })
             }
             PacketC::ClearProgress { item } => {
-                if let Some(slot) = self.get_item(item) {
-                    if let Some(item) = slot {
-                        item.active = None;
-                    }
+                if let Some(Some(item)) = self.get_item(item) {
+                    item.active = None;
                 }
             }
             PacketC::SetProgress {
@@ -373,16 +369,14 @@ impl Game {
                 player,
                 warn,
             } => {
-                if let Some(slot) = self.get_item(item) {
-                    if let Some(item) = slot {
-                        item.active = Some(Involvement {
-                            position,
-                            speed,
-                            player,
-                            warn,
-                            recipe: RecipeIndex(0),
-                        });
-                    }
+                if let Some(Some(item)) = self.get_item(item) {
+                    item.active = Some(Involvement {
+                        position,
+                        speed,
+                        player,
+                        warn,
+                        recipe: RecipeIndex(0),
+                    });
                 }
             }
             PacketC::ServerMessage { .. } => {
@@ -517,10 +511,8 @@ impl Player {
                 _ => (),
             }
         }
-        for item in &self.items {
-            if let Some(item) = item {
-                item.draw(ctx, item_sprites)
-            }
+        for item in self.items.iter().flatten() {
+            item.draw(ctx, item_sprites)
         }
     }
 }