fixup! fixup! chore: naively set warn(unsafe_op_in_unsafe_fn) in wgpu-hal

Author: ErichDonGubler

wgpu-hal/src/dx11/device.rs

@@ -227,16 +227,16 @@ impl crate::Queue<super::Api> for super::Queue {
 impl super::D3D11Device {
     #[allow(trivial_casts)] // come on
     pub unsafe fn check_feature_support<T>(&self, feature: d3d11::D3D11_FEATURE) -> T {
-        let mut value = unsafe { mem::zeroed::<T>() };
-        let ret = unsafe {
-            self.CheckFeatureSupport(
+        unsafe {
+            let mut value = mem::zeroed::<T>();
+            let ret = self.CheckFeatureSupport(
                 feature,
                 &mut value as *mut T as *mut c_void,
                 mem::size_of::<T>() as u32,
-            )
-        };
-        assert_eq!(ret.into_result(), Ok(()));
+            );
+            assert_eq!(ret.into_result(), Ok(()));
 
-        value
+            value
+        }
     }
 }

wgpu-hal/src/dx12/mod.rs

@@ -195,9 +195,11 @@ struct CommandSignatures {
 
 impl CommandSignatures {
     unsafe fn destroy(&self) {
-        unsafe { self.draw.destroy() };
-        unsafe { self.draw_indexed.destroy() };
-        unsafe { self.dispatch.destroy() };
+        unsafe {
+            self.draw.destroy();
+            self.draw_indexed.destroy();
+            self.dispatch.destroy();
+        }
     }
 }
 
@@ -210,10 +212,12 @@ struct DeviceShared {
 
 impl DeviceShared {
     unsafe fn destroy(&self) {
-        unsafe { self.zero_buffer.destroy() };
-        unsafe { self.cmd_signatures.destroy() };
-        unsafe { self.heap_views.raw.destroy() };
-        unsafe { self.heap_samplers.raw.destroy() };
+        unsafe {
+            self.zero_buffer.destroy();
+            self.cmd_signatures.destroy();
+            self.heap_views.raw.destroy();
+            self.heap_samplers.raw.destroy();
+        }
     }
 }
 
@@ -730,12 +734,14 @@ impl crate::Surface<Api> for Surface {
 
     unsafe fn unconfigure(&mut self, device: &Device) {
         if let Some(mut sc) = self.swap_chain.take() {
-            let _ = unsafe { sc.wait(None) };
-            //TODO: this shouldn't be needed,
-            // but it complains that the queue is still used otherwise
-            let _ = unsafe { device.wait_idle() };
-            let raw = unsafe { sc.release_resources() };
-            unsafe { raw.destroy() };
+            unsafe {
+                let _ = sc.wait(None);
+                //TODO: this shouldn't be needed,
+                // but it complains that the queue is still used otherwise
+                let _ = device.wait_idle();
+                let raw = sc.release_resources();
+                raw.destroy();
+            }
         }
     }