From 9695c361104859d26513ef642c27a1d1bf3b4cb2 Mon Sep 17 00:00:00 2001
From: Ivor Wanders <ivor@iwanders.net>
Date: Sun, 22 Sep 2024 12:45:18 -0400
Subject: [PATCH] Fix clippy 1.81 warnings.

---
 battleground_construct/src/units/common.rs          |  2 +-
 battleground_unit_control/src/register_interface.rs |  6 ++----
 battleground_viewer/src/lib.rs                      | 10 +++++-----
 3 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/battleground_construct/src/units/common.rs b/battleground_construct/src/units/common.rs
index e84874f..b1cd676 100644
--- a/battleground_construct/src/units/common.rs
+++ b/battleground_construct/src/units/common.rs
@@ -257,7 +257,7 @@ impl ComponentBox {
             };
 
             let error = setpoint - revolute.position();
-            revolute.set_velocity_cmd(error.min(0.3).max(-0.3));
+            revolute.set_velocity_cmd(error.clamp(-0.3, 0.3));
 
             if error.abs() < 0.05 {
                 revolute.set_velocity_cmd(0.0);
diff --git a/battleground_unit_control/src/register_interface.rs b/battleground_unit_control/src/register_interface.rs
index 3ccd10b..aacabb3 100644
--- a/battleground_unit_control/src/register_interface.rs
+++ b/battleground_unit_control/src/register_interface.rs
@@ -81,7 +81,7 @@ impl Register {
             value: Value::Bytes {
                 values: vec![],
                 min_len: 0,
-                max_len: std::usize::MAX,
+                max_len: usize::MAX,
             },
         }
     }
@@ -244,9 +244,7 @@ impl RegisterInterface {
         );
     }
 
-    pub fn add_module<M: UnitModule>(&mut self, name: &str, index: ModuleId, handler: M)
-    where
-        M: Sized + 'static,
+    pub fn add_module<M: UnitModule + Sized + 'static>(&mut self, name: &str, index: ModuleId, handler: M)
     {
         self.modules.insert(
             index,
diff --git a/battleground_viewer/src/lib.rs b/battleground_viewer/src/lib.rs
index ae01cbd..ed1f7ea 100644
--- a/battleground_viewer/src/lib.rs
+++ b/battleground_viewer/src/lib.rs
@@ -332,14 +332,14 @@ impl ConstructViewer {
                 .clear(ClearState::color_and_depth(0.8, 0.8, 0.8, 1.0, 1.0))
                 .render(
                     &self.camera,
-                    &self.construct_render.objects(RenderPass::BaseScene),
+                    self.construct_render.objects(RenderPass::BaseScene),
                     &[&self.ambient_light, &self.directional_light],
                 );
             screen
                 // .clear(ClearState::color_and_depth(0.8, 0.8, 0.8, 1.0, 1.0))
                 .render(
                     &self.camera,
-                    &self.construct_render.objects(RenderPass::BaseSceneOverlay),
+                    self.construct_render.objects(RenderPass::BaseSceneOverlay),
                     &[&self.ambient_light, &self.directional_light],
                 );
             // B1) Render depth buffer with non-emissives
@@ -364,7 +364,7 @@ impl ConstructViewer {
                 .render_with_material(
                     &write_depth_material,
                     &self.camera,
-                    &self.construct_render.objects(RenderPass::NonGlowDepths),
+                    self.construct_render.objects(RenderPass::NonGlowDepths),
                     &[],
                 );
 
@@ -386,7 +386,7 @@ impl ConstructViewer {
             )
             .render(
                 &self.camera,
-                &self.construct_render.objects(RenderPass::GlowSources),
+                self.construct_render.objects(RenderPass::GlowSources),
                 &[],
             );
 
@@ -395,7 +395,7 @@ impl ConstructViewer {
             screen.render_with_material(
                 &fence_material,
                 &self.camera,
-                &self.construct_render.objects(RenderPass::Fences),
+                self.construct_render.objects(RenderPass::Fences),
                 &[],
             );