coderabbit

Author: richiemcilroy

crates/frame-converter/src/d3d11.rs

@@ -498,7 +498,7 @@ impl FrameConverter for D3D11Converter {
 fn pixel_to_dxgi(pixel: Pixel) -> Result<DXGI_FORMAT, ConvertError> {
     match pixel {
         Pixel::NV12 => Ok(DXGI_FORMAT_NV12),
-        Pixel::YUYV422 => Ok(DXGI_FORMAT_YUY2),
+        Pixel::YUYV422 | Pixel::UYVY422 => Ok(DXGI_FORMAT_YUY2),
         _ => Err(ConvertError::UnsupportedFormat(pixel, Pixel::NV12)),
     }
 }
@@ -548,35 +548,29 @@ unsafe fn copy_frame_to_mapped(frame: &frame::Video, dst: *mut u8, dst_stride: u
     match format {
         Pixel::NV12 => {
             for y in 0..height {
-                unsafe {
-                    ptr::copy_nonoverlapping(
-                        frame.data(0).as_ptr().add(y * frame.stride(0)),
-                        dst.add(y * dst_stride),
-                        frame.width() as usize,
-                    );
-                }
+                ptr::copy_nonoverlapping(
+                    frame.data(0).as_ptr().add(y * frame.stride(0)),
+                    dst.add(y * dst_stride),
+                    frame.width() as usize,
+                );
             }
             let uv_offset = height * dst_stride;
             for y in 0..height / 2 {
-                unsafe {
-                    ptr::copy_nonoverlapping(
-                        frame.data(1).as_ptr().add(y * frame.stride(1)),
-                        dst.add(uv_offset + y * dst_stride),
-                        frame.width() as usize,
-                    );
-                }
+                ptr::copy_nonoverlapping(
+                    frame.data(1).as_ptr().add(y * frame.stride(1)),
+                    dst.add(uv_offset + y * dst_stride),
+                    frame.width() as usize,
+                );
             }
         }
         Pixel::YUYV422 | Pixel::UYVY422 => {
             let row_bytes = frame.width() as usize * 2;
             for y in 0..height {
-                unsafe {
-                    ptr::copy_nonoverlapping(
-                        frame.data(0).as_ptr().add(y * frame.stride(0)),
-                        dst.add(y * dst_stride),
-                        row_bytes,
-                    );
-                }
+                ptr::copy_nonoverlapping(
+                    frame.data(0).as_ptr().add(y * frame.stride(0)),
+                    dst.add(y * dst_stride),
+                    row_bytes,
+                );
             }
         }
         _ => {}
@@ -590,36 +584,30 @@ unsafe fn copy_mapped_to_frame(src: *const u8, src_stride: usize, frame: &mut fr
     match format {
         Pixel::NV12 => {
             for y in 0..height {
-                unsafe {
-                    ptr::copy_nonoverlapping(
-                        src.add(y * src_stride),
-                        frame.data_mut(0).as_mut_ptr().add(y * frame.stride(0)),
-                        frame.width() as usize,
-                    );
-                }
+                ptr::copy_nonoverlapping(
+                    src.add(y * src_stride),
+                    frame.data_mut(0).as_mut_ptr().add(y * frame.stride(0)),
+                    frame.width() as usize,
+                );
             }
             let uv_offset = height * src_stride;
             for y in 0..height / 2 {
-                unsafe {
-                    ptr::copy_nonoverlapping(
-                        src.add(uv_offset + y * src_stride),
-                        frame.data_mut(1).as_mut_ptr().add(y * frame.stride(1)),
-                        frame.width() as usize,
-                    );
-                }
+                ptr::copy_nonoverlapping(
+                    src.add(uv_offset + y * src_stride),
+                    frame.data_mut(1).as_mut_ptr().add(y * frame.stride(1)),
+                    frame.width() as usize,
+                );
             }
         }
-        Pixel::YUYV422 => {
+        Pixel::YUYV422 | Pixel::UYVY422 => {
             let bytes_per_pixel = 2;
             let row_bytes = frame.width() as usize * bytes_per_pixel;
             for y in 0..height {
-                unsafe {
-                    ptr::copy_nonoverlapping(
-                        src.add(y * src_stride),
-                        frame.data_mut(0).as_mut_ptr().add(y * frame.stride(0)),
-                        row_bytes,
-                    );
-                }
+                ptr::copy_nonoverlapping(
+                    src.add(y * src_stride),
+                    frame.data_mut(0).as_mut_ptr().add(y * frame.stride(0)),
+                    row_bytes,
+                );
             }
         }
         _ => {}