Rename BorrowedCursor::advance_unchecked to advance

Author: a1phyr

library/core/src/io/borrowed_buf.rs

@@ -272,7 +272,7 @@ impl<'a> BorrowedCursor<'a> {
     /// Panics if there are less than `n` bytes initialized.
     #[unstable(feature = "borrowed_buf_init", issue = "78485")]
     #[inline]
-    pub fn advance(&mut self, n: usize) -> &mut Self {
+    pub fn advance_checked(&mut self, n: usize) -> &mut Self {
         // The substraction cannot underflow by invariant of this type.
         let init_unfilled = if self.buf.init { self.buf.buf.len() - self.buf.filled } else { 0 };
         assert!(n <= init_unfilled);
@@ -292,7 +292,7 @@ impl<'a> BorrowedCursor<'a> {
     /// The caller must ensure that the first `n` bytes of the cursor have been properly
     /// initialised.
     #[inline]
-    pub unsafe fn advance_unchecked(&mut self, n: usize) -> &mut Self {
+    pub unsafe fn advance(&mut self, n: usize) -> &mut Self {
         self.buf.filled += n;
         self
     }

library/coretests/tests/io/borrowed_buf.rs

@@ -40,7 +40,7 @@ fn advance_filled() {
     let buf: &mut [_] = &mut [0; 16];
     let mut rbuf: BorrowedBuf<'_> = buf.into();
 
-    rbuf.unfilled().advance(1);
+    rbuf.unfilled().advance_checked(1);
 
     assert_eq!(rbuf.filled().len(), 1);
     assert_eq!(rbuf.unfilled().capacity(), 15);
@@ -51,7 +51,7 @@ fn clear() {
     let buf: &mut [_] = &mut [255; 16];
     let mut rbuf: BorrowedBuf<'_> = buf.into();
 
-    rbuf.unfilled().advance(16);
+    rbuf.unfilled().advance_checked(16);
 
     assert_eq!(rbuf.filled().len(), 16);
     assert_eq!(rbuf.unfilled().capacity(), 0);
@@ -131,7 +131,7 @@ fn cursor_set_init() {
     assert!(cursor.is_init());
     assert_eq!(unsafe { cursor.as_mut().len() }, 16);
 
-    cursor.advance(4);
+    cursor.advance_checked(4);
 
     assert_eq!(unsafe { cursor.as_mut().len() }, 12);
 
@@ -157,7 +157,7 @@ fn cursor_with_unfilled_buf() {
         assert!(!buf.is_init());
 
         buf.unfilled().ensure_init();
-        buf.unfilled().advance(4);
+        buf.unfilled().advance_checked(4);
     });
 
     assert!(cursor.is_init());

library/std/src/io/mod.rs

@@ -570,7 +570,7 @@ where
     F: FnOnce(&mut [u8]) -> Result<usize>,
 {
     let n = read(cursor.ensure_init())?;
-    cursor.advance(n);
+    cursor.advance_checked(n);
     Ok(())
 }
 
@@ -3113,7 +3113,7 @@ impl<T: Read> Read for Take<T> {
 
             unsafe {
                 // SAFETY: filled bytes have been filled
-                buf.advance_unchecked(filled);
+                buf.advance(filled);
             }
 
             self.limit -= filled as u64;

library/std/src/io/util.rs

@@ -283,7 +283,7 @@ impl Read for Repeat {
         // SAFETY: No uninit bytes are being written.
         unsafe { buf.as_mut() }.write_filled(self.byte);
         // SAFETY: the entire unfilled portion of buf has been initialized.
-        unsafe { buf.advance_unchecked(buf.capacity()) };
+        unsafe { buf.advance(buf.capacity()) };
         Ok(())
     }
 

library/std/src/sys/fd/hermit.rs

@@ -33,7 +33,7 @@ impl FileDesc {
             )
         })?;
         // SAFETY: Exactly `result` bytes have been filled.
-        unsafe { buf.advance_unchecked(result as usize) };
+        unsafe { buf.advance(result as usize) };
         Ok(())
     }
 

library/std/src/sys/fd/unix.rs

@@ -185,7 +185,7 @@ impl FileDesc {
 
         // SAFETY: `ret` bytes were written to the initialized portion of the buffer
         unsafe {
-            cursor.advance_unchecked(ret as usize);
+            cursor.advance(ret as usize);
         }
         Ok(())
     }
@@ -203,7 +203,7 @@ impl FileDesc {
 
         // SAFETY: `ret` bytes were written to the initialized portion of the buffer
         unsafe {
-            cursor.advance_unchecked(ret as usize);
+            cursor.advance(ret as usize);
         }
         Ok(())
     }

library/std/src/sys/fs/solid.rs

@@ -401,7 +401,7 @@ impl File {
 
             // Safety: `num_bytes_read` bytes were written to the unfilled
             // portion of the buffer
-            cursor.advance_unchecked(num_bytes_read);
+            cursor.advance(num_bytes_read);
 
             Ok(())
         }

library/std/src/sys/net/connection/socket/hermit.rs

@@ -143,7 +143,7 @@ impl Socket {
             )
         })?;
         unsafe {
-            buf.advance_unchecked(ret as usize);
+            buf.advance(ret as usize);
         }
         Ok(())
     }

library/std/src/sys/net/connection/socket/solid.rs

@@ -190,7 +190,7 @@ impl Socket {
             netc::recv(self.as_raw_fd(), buf.as_mut().as_mut_ptr().cast(), buf.capacity(), flags)
         })?;
         unsafe {
-            buf.advance_unchecked(ret as usize);
+            buf.advance(ret as usize);
         }
         Ok(())
     }

library/std/src/sys/net/connection/socket/unix.rs

@@ -294,7 +294,7 @@ impl Socket {
             )
         })?;
         unsafe {
-            buf.advance_unchecked(ret as usize);
+            buf.advance(ret as usize);
         }
         Ok(())
     }