fixup! address comments from viirya

Author: klion26

parquet-variant/src/builder.rs

@@ -64,9 +64,9 @@ fn write_offset(buf: &mut Vec<u8>, value: usize, nbytes: u8) {
 }
 
 /// Write little-endian integer to buffer at a specific position
-fn write_offset_at_pos(buf: &mut [u8], index: usize, value: usize, nbytes: u8) {
+fn write_offset_at_pos(buf: &mut [u8], start_pos: usize, value: usize, nbytes: u8) {
     let bytes = value.to_le_bytes();
-    buf[index..index + nbytes as usize].copy_from_slice(&bytes[..nbytes as usize]);
+    buf[start_pos..start_pos + nbytes as usize].copy_from_slice(&bytes[..nbytes as usize]);
 }
 
 /// Wrapper around a `Vec<u8>` that provides methods for appending
@@ -383,15 +383,14 @@ impl ValueBuffer {
     /// from the starting position of the buffer, will return the position after this write
     fn append_offset_array_start_from_buf_pos(
         &mut self,
-        start_index: usize,
+        start_pos: usize,
         offsets: impl IntoIterator<Item = usize>,
         data_size: Option<usize>,
         nbytes: u8,
     ) -> usize {
         let buf = self.inner_mut();
 
-        // Write field offsets (adjusted for header)
-        let mut current_pos = start_index; // skip the header byte
+        let mut current_pos = start_pos;
         for relative_offset in offsets {
             write_offset_at_pos(buf, current_pos, relative_offset, nbytes);
             current_pos += nbytes as usize;