feat(io): adjust read_managed_at param order (#431)

Author: Berrysoft

compio-fs/src/file.rs

@@ -175,9 +175,9 @@ impl AsyncReadManagedAt for File {
 
     async fn read_managed_at<'a>(
         &self,
-        pos: u64,
         buffer_pool: &'a Self::BufferPool,
         len: usize,
+        pos: u64,
     ) -> io::Result<Self::Buffer<'a>> {
         let fd = self.inner.to_shared_fd();
         let buffer_pool = buffer_pool.try_inner()?;

compio-fs/src/named_pipe.rs

@@ -384,7 +384,7 @@ impl AsyncReadManaged for &NamedPipeClient {
         len: usize,
     ) -> io::Result<Self::Buffer<'a>> {
         // The position is ignored.
-        self.handle.read_managed_at(0, buffer_pool, len).await
+        self.handle.read_managed_at(buffer_pool, len, 0).await
     }
 }
 

compio-fs/tests/buffer_pool.rs

@@ -22,7 +22,7 @@ async fn test_read_file() {
 
     let file = File::open(tempfile.path()).await.unwrap();
     let buffer_pool = BufferPool::new(1, 15).unwrap();
-    let buf = file.read_managed_at(0, &buffer_pool, 0).await.unwrap();
+    let buf = file.read_managed_at(&buffer_pool, 0, 0).await.unwrap();
 
     assert_eq!(buf.len(), HELLO.len());
     assert_eq!(buf.as_ref(), HELLO);

compio-io/src/read/managed.rs

@@ -41,9 +41,9 @@ pub trait AsyncReadManagedAt {
     /// read max len
     async fn read_managed_at<'a>(
         &self,
-        pos: u64,
         buffer_pool: &'a Self::BufferPool,
         len: usize,
+        pos: u64,
     ) -> IoResult<Self::Buffer<'a>>;
 }
 
@@ -62,7 +62,7 @@ where
         let pos = self.position();
         let buf = self
             .get_ref()
-            .read_managed_at(pos, buffer_pool, len)
+            .read_managed_at(buffer_pool, len, pos)
             .await?;
         self.set_position(pos + buf.len() as u64);
         Ok(buf)