From cff2575a32219d4294ab05d69ad37eb6bd7a7b2d Mon Sep 17 00:00:00 2001
From: HampusM <hampus@hampusmat.com>
Date: Wed, 21 Aug 2024 20:38:41 +0200
Subject: refactor(engine): fix clippy lints

---
 engine/src/renderer/opengl.rs | 42 ++++++++++++++++++++++--------------------
 1 file changed, 22 insertions(+), 20 deletions(-)

(limited to 'engine/src/renderer')

diff --git a/engine/src/renderer/opengl.rs b/engine/src/renderer/opengl.rs
index 292fd94..888f760 100644
--- a/engine/src/renderer/opengl.rs
+++ b/engine/src/renderer/opengl.rs
@@ -111,6 +111,7 @@ fn initialize(window: Single<Window>)
     enable(Capability::MultiSample);
 }
 
+#[allow(clippy::too_many_arguments)]
 fn render(
     query: Query<RenderableEntity, Not<With<NoDraw>>>,
     point_light_query: Query<(PointLight,)>,
@@ -166,21 +167,18 @@ fn render(
 
         let new_gl_objects;
 
-        let gl_objects = match gl_objects.as_deref() {
-            Some(gl_objects) => gl_objects,
-            None => {
-                // TODO: Account for when meshes are changed
-                let gl_objects = GlObjects::new(&*mesh);
+        let gl_objects = if let Some(gl_objects) = gl_objects.as_deref() {
+            gl_objects
+        } else {
+            // TODO: Account for when meshes are changed
+            let gl_objects = GlObjects::new(&mesh);
 
-                new_gl_objects = Some(gl_objects.clone());
+            new_gl_objects = Some(gl_objects.clone());
 
-                actions.add_components(
-                    query.entity_uid(entity_index).unwrap(),
-                    (gl_objects,),
-                );
+            actions
+                .add_components(query.entity_uid(entity_index).unwrap(), (gl_objects,));
 
-                &*new_gl_objects.unwrap()
-            }
+            &*new_gl_objects.unwrap()
         };
 
         apply_transformation_matrices(
@@ -229,10 +227,10 @@ fn render(
             crate::opengl::set_polygon_mode(
                 draw_flags.polygon_mode_config.face,
                 draw_flags.polygon_mode_config.mode,
-            )
+            );
         }
 
-        draw_mesh(&gl_objects);
+        draw_mesh(gl_objects);
 
         if draw_flags.is_some() {
             let default_polygon_mode_config = PolygonModeConfig::default();
@@ -240,7 +238,7 @@ fn render(
             crate::opengl::set_polygon_mode(
                 default_polygon_mode_config.face,
                 default_polygon_mode_config.mode,
-            )
+            );
         }
     }
 }
@@ -332,7 +330,7 @@ fn create_gl_shader_program(
 struct GlObjects
 {
     /// Vertex and index buffer has to live as long as the vertex array
-    _vertex_buffer: Buffer<Vertex>,
+    vertex_buffer: Buffer<Vertex>,
     index_buffer: Option<Buffer<u32>>,
     index_cnt: u32,
 
@@ -388,7 +386,7 @@ impl GlObjects
             vertex_arr.bind_element_buffer(&index_buffer);
 
             return Self {
-                _vertex_buffer: vertex_buffer,
+                vertex_buffer,
                 index_buffer: Some(index_buffer),
                 index_cnt: indices.len().try_into().unwrap(),
                 vertex_arr,
@@ -396,7 +394,7 @@ impl GlObjects
         }
 
         Self {
-            _vertex_buffer: vertex_buffer,
+            vertex_buffer,
             index_buffer: None,
             index_cnt: 0,
             vertex_arr,
@@ -407,7 +405,7 @@ impl GlObjects
     {
         NeverDrop::new(Self {
             // SAFETY: The vertex buffer will never become dropped (NeverDrop ensures it)
-            _vertex_buffer: unsafe { self._vertex_buffer.clone_weak() },
+            vertex_buffer: unsafe { self.vertex_buffer.clone_weak() },
             index_buffer: self
                 .index_buffer
                 .as_ref()
@@ -488,6 +486,8 @@ fn apply_light<PointLightHolder>(
         );
     }
 
+    // There probably won't be more than 2147483648 directional lights
+    #[allow(clippy::cast_possible_wrap, clippy::cast_possible_truncation)]
     gl_shader_program.set_uniform_1i(
         cstr!("directional_light_cnt"),
         directional_lights.len() as i32,
@@ -514,6 +514,8 @@ fn apply_light<PointLightHolder>(
         );
     }
 
+    // There probably won't be more than 2147483648 point lights
+    #[allow(clippy::cast_possible_wrap, clippy::cast_possible_truncation)]
     gl_shader_program.set_uniform_1i(cstr!("point_light_cnt"), point_lights.len() as i32);
 
     gl_shader_program.set_uniform_vec_3fv(
@@ -521,7 +523,7 @@ fn apply_light<PointLightHolder>(
         &if material_flags.use_ambient_color {
             material.ambient.clone()
         } else {
-            global_light.ambient.clone().into()
+            global_light.ambient.clone()
         }
         .into(),
     );
-- 
cgit v1.2.3-18-g5258