naming

Signed-off-by: Alexander Droste <[email protected]>

Author: 0ax1

vortex-compute/src/expand/buffer.rs

@@ -26,7 +26,7 @@ impl<T: Copy> Expand for Buffer<T> {
                         (&mut buf_mut).expand(mask);
                         buf_mut.freeze()
                     }
-                    // Otherwise, expand into a new buffer at the target size.
+                    // Otherwise, expand into a new buffer.
                     Err(buffer) => expand_into_new_buffer(buffer.as_slice(), mask),
                 }
             }
@@ -83,7 +83,7 @@ impl<T: Copy> Expand for &mut BufferMut<T> {
                 }
 
                 let buf_slice = self.as_mut_slice();
-                scatter_into_slice(buf_slice, buf_len, mask_values);
+                expand_into_slice_inplace(buf_slice, buf_len, mask_values);
             }
         }
     }
@@ -97,7 +97,7 @@ impl<T: Copy> Expand for &mut BufferMut<T> {
 /// * `buf_slice` - The buffer slice to scatter into (already expanded to mask length)
 /// * `src_len` - The original length of the buffer before expansion
 /// * `mask_values` - The mask indicating where elements should be placed
-fn scatter_into_slice<T: Copy>(
+fn expand_into_slice_inplace<T: Copy>(
     buf_slice: &mut [T],
     src_len: usize,
     mask_values: &vortex_mask::MaskValues,
@@ -136,13 +136,13 @@ fn scatter_into_slice<T: Copy>(
 ///
 /// # Arguments
 ///
-/// * `dest` - The destination buffer slice (already expanded to mask length)
 /// * `src` - The source elements to scatter
+/// * `dest` - The destination buffer slice (already expanded to mask length)
 /// * `src_len` - The length of the source buffer
 /// * `mask_values` - The mask indicating where elements should be placed
 fn scatter_into_slice_from<T: Copy>(
-    dest: &mut [T],
     src: &[T],
+    dest: &mut [T],
     src_len: usize,
     mask_values: &vortex_mask::MaskValues,
 ) {
@@ -204,7 +204,7 @@ fn expand_into_new_buffer<T: Copy>(src: &[T], mask: &Mask) -> Buffer<T> {
             }
 
             let buf_slice = buf_mut.as_mut_slice();
-            scatter_into_slice_from(buf_slice, src, src_len, mask_values);
+            scatter_into_slice_from(src, buf_slice, src_len, mask_values);
             buf_mut.freeze()
         }
     }