From 17b67fa997bc9614171631efa9f6bbe13663a678 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Crozet?= <sebcrozet@dimforge.com>
Date: Sat, 23 Mar 2024 10:25:56 +0100
Subject: [PATCH] chore: clippy fix

---
 src_testbed/physics/mod.rs | 44 +++++++++++++++++++-------------------
 src_testbed/testbed.rs     |  6 +++---
 2 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/src_testbed/physics/mod.rs b/src_testbed/physics/mod.rs
index bc003271a..c3b750fe1 100644
--- a/src_testbed/physics/mod.rs
+++ b/src_testbed/physics/mod.rs
@@ -18,6 +18,17 @@ pub struct PhysicsSnapshot {
     island_manager: Vec<u8>,
 }
 
+pub struct DeserializedPhysicsSnapshot {
+    pub timestep_id: usize,
+    pub broad_phase: BroadPhase,
+    pub narrow_phase: NarrowPhase,
+    pub island_manager: IslandManager,
+    pub bodies: RigidBodySet,
+    pub colliders: ColliderSet,
+    pub impulse_joints: ImpulseJointSet,
+    pub multibody_joints: MultibodyJointSet,
+}
+
 impl PhysicsSnapshot {
     pub fn new(
         timestep_id: usize,
@@ -41,28 +52,17 @@ impl PhysicsSnapshot {
         })
     }
 
-    pub fn restore(
-        &self,
-    ) -> bincode::Result<(
-        usize,
-        BroadPhase,
-        NarrowPhase,
-        IslandManager,
-        RigidBodySet,
-        ColliderSet,
-        ImpulseJointSet,
-        MultibodyJointSet,
-    )> {
-        Ok((
-            self.timestep_id,
-            bincode::deserialize(&self.broad_phase)?,
-            bincode::deserialize(&self.narrow_phase)?,
-            bincode::deserialize(&self.island_manager)?,
-            bincode::deserialize(&self.bodies)?,
-            bincode::deserialize(&self.colliders)?,
-            bincode::deserialize(&self.impulse_joints)?,
-            bincode::deserialize(&self.multibody_joints)?,
-        ))
+    pub fn restore(&self) -> bincode::Result<DeserializedPhysicsSnapshot> {
+        Ok(DeserializedPhysicsSnapshot {
+            timestep_id: self.timestep_id,
+            broad_phase: bincode::deserialize(&self.broad_phase)?,
+            narrow_phase: bincode::deserialize(&self.narrow_phase)?,
+            island_manager: bincode::deserialize(&self.island_manager)?,
+            bodies: bincode::deserialize(&self.bodies)?,
+            colliders: bincode::deserialize(&self.colliders)?,
+            impulse_joints: bincode::deserialize(&self.impulse_joints)?,
+            multibody_joints: bincode::deserialize(&self.multibody_joints)?,
+        })
     }
 
     pub fn print_snapshot_len(&self) {
diff --git a/src_testbed/testbed.rs b/src_testbed/testbed.rs
index 543038457..780e23ee5 100644
--- a/src_testbed/testbed.rs
+++ b/src_testbed/testbed.rs
@@ -7,7 +7,7 @@ use std::num::NonZeroUsize;
 use bevy::prelude::*;
 
 use crate::debug_render::{DebugRenderPipelineResource, RapierDebugRenderPlugin};
-use crate::physics::{PhysicsEvents, PhysicsSnapshot, PhysicsState};
+use crate::physics::{DeserializedPhysicsSnapshot, PhysicsEvents, PhysicsSnapshot, PhysicsState};
 use crate::plugin::TestbedPlugin;
 use crate::ui;
 use crate::{graphics::GraphicsManager, harness::RunState};
@@ -1271,7 +1271,7 @@ fn update_testbed(
                 .action_flags
                 .set(TestbedActionFlags::RESTORE_SNAPSHOT, false);
             if let Some(snapshot) = &state.snapshot {
-                if let Ok((
+                if let Ok(DeserializedPhysicsSnapshot {
                     timestep_id,
                     broad_phase,
                     narrow_phase,
@@ -1280,7 +1280,7 @@ fn update_testbed(
                     colliders,
                     impulse_joints,
                     multibody_joints,
-                )) = snapshot.restore()
+                }) = snapshot.restore()
                 {
                     clear(&mut commands, &mut state, &mut graphics, &mut plugins);