Address review and small fixups

Remove pattern of using slicing to do one bounds check now that arrays
are being used.
Break array::map function into its own line
Write directly into array instead of indexing since the types match
Remove no-op `max(0)` call on a usize

Author: okaneco

src/codecs/avif/decoder.rs

@@ -516,9 +516,7 @@ impl<R: Read> ImageDecoder for AvifDecoder<R> {
                 self.process_16bit_picture(&mut aligned_store, yuv_range, matrix_strategy)?;
                 let buf_chunks = buf.as_chunks_mut::<2>().0.iter_mut();
                 for (dst, src) in buf_chunks.zip(aligned_store.iter()) {
-                    let bytes = src.to_ne_bytes();
-                    dst[0] = bytes[0];
-                    dst[1] = bytes[1];
+                    *dst = src.to_ne_bytes();
                 }
             }
         }

src/imageops/fast_blur.rs

@@ -332,11 +332,10 @@ fn box_blur_horizontal_pass_impl<T, const CN: usize>(
             let dst_chunks = dst.as_chunks_mut::<CN>().0.iter_mut();
             let data_section_chunks = data_section.as_chunks::<CN>().0.iter();
             let advanced_kernel_part_chunks = advanced_kernel_part.as_chunks::<CN>().0.iter();
-            for ((dst, src_previous), src_next) in dst_chunks
+            for ((dst_chunk, src_previous), src_next) in dst_chunks
                 .zip(data_section_chunks)
                 .zip(advanced_kernel_part_chunks)
             {
-                let dst_chunk = &mut dst[..CN];
                 dst_chunk[0] = rounding_saturating_mul(weight0, weight);
                 if CN > 1 {
                     dst_chunk[1] = rounding_saturating_mul(weight1, weight);

src/imageops/filter_1d.rs

@@ -100,7 +100,7 @@ where
     for (x, dst) in (image_size.width..(image_size.width + pad_w))
         .zip(row_buffer.as_chunks_mut::<N>().0.iter_mut().rev())
     {
-        let old_x = x.max(0).min(image_size.width - 1);
+        let old_x = x.min(image_size.width - 1);
         let old_px = old_x * N;
         let src_iter = &source_row[old_px..(old_px + N)];
         for (dst, src) in dst.iter_mut().zip(src_iter.iter()) {
@@ -269,14 +269,12 @@ fn filter_symmetric_column<T, F, const N: usize>(
                 }
             }
 
-            let shaped_dst0 = &mut chunk[..16];
+            let (shaped_dst0, shaped_dst1) = chunk.split_at_mut(16);
 
             for (src, dst) in store0.iter().zip(shaped_dst0.iter_mut()) {
                 *dst = src.to_();
             }
 
-            let shaped_dst1 = &mut chunk[16..32];
-
             for (src, dst) in store1.iter().zip(shaped_dst1.iter_mut()) {
                 *dst = src.to_();
             }

src/imageops/sample.rs

@@ -1512,7 +1512,8 @@ where
     let mut out = image.buffer_like();
     let transient_dst_chunks = transient_dst.as_chunks_mut::<CN>().0.iter_mut();
     for (dst, src) in out.pixels_mut().zip(transient_dst_chunks) {
-        *dst = *Pixel::from_slice(&src.map(|v| NumCast::from(FloatNearest(v)).unwrap()))
+        let pix = src.map(|v| NumCast::from(FloatNearest(v)).unwrap());
+        *dst = *Pixel::from_slice(&pix);
     }
 
     out