Clippy fixes

src/core/instance.rs

@@ -28,9 +28,6 @@ impl Vertex for InstanceRaw {
         use std::mem;
         wgpu::VertexBufferLayout {
             array_stride: mem::size_of::<InstanceRaw>() as wgpu::BufferAddress,
-            // We need to switch from using a step mode of Vertex to Instance
-            // This means that our shaders will only change to use the next
-            // instance when the shader starts processing a new instance
             step_mode: wgpu::VertexStepMode::Instance,
             attributes: &[
                 // model matrix

src/core/light.rs

@@ -33,7 +33,7 @@ impl LightUniform {
             position,
             _padding: 0,
             color,
-            matrices: matrices,
+            matrices,
         }
     }
 }

src/core/pass.rs

@@ -22,8 +22,8 @@ impl RenderPass {
     ) -> Self {
         let layout = device.create_pipeline_layout(&wgpu::PipelineLayoutDescriptor {
             label: Some((label.to_owned() + " pipeline Layout").as_str()),
-            bind_group_layouts: bind_group_layouts,
+            bind_group_layouts,
-            push_constant_ranges: push_constant_ranges,
+            push_constant_ranges,
         });
         let shader = wgpu::ShaderModuleDescriptor {
             label: Some(shader_name),
@@ -80,7 +80,7 @@ impl RenderPass {
                 entry_point: "vs_main",
                 buffers: vertex_layouts,
             },
-            fragment: fragment,
+            fragment,
             primitive: wgpu::PrimitiveState {
                 topology: wgpu::PrimitiveTopology::TriangleList,
                 strip_index_format: None,

src/core/state.rs

@@ -165,7 +165,7 @@ impl State {
                     aspect: wgpu::TextureAspect::All,
                     base_mip_level: 0,
                     mip_level_count: None,
-                    base_array_layer: i as u32,
+                    base_array_layer: i,
                     array_layer_count: Some(1),
                 })
             })

src/core/texture.rs

@@ -20,8 +20,8 @@ impl Texture {
         usage: wgpu::TextureUsages,
     ) -> Self {
         let size = wgpu::Extent3d {
-            width: width,
+            width,
-            height: height,
+            height,
             depth_or_array_layers: layers,
         };
         let desc = wgpu::TextureDescriptor {

src/core/window.rs

@@ -120,15 +120,14 @@ pub async fn run() {
 
 fn lock_cursor(window: &winit::window::Window, lock: bool) {
     if lock {
-        match window
+        if let Err(e) = window
             .set_cursor_grab(if cfg!(target_arch = "wasm32") {
                 winit::window::CursorGrabMode::Locked
             } else {
                 winit::window::CursorGrabMode::Confined
             })
         {
-            Err(e) => println!("Failed to grab cursor {e:?}"),
+            println!("Failed to grab cursor {e:?}")
-            _ => ()
         }
         window.set_cursor_visible(false);
     } else {