prefer to directly return .last_os_error, vs conversion

rustcoreutils · Mar 23, 2024 · 6927182 · 6927182
1 parent 246af53
commit 6927182
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 28 deletions.
diff --git a/fs/src/df.rs b/fs/src/df.rs
@@ -57,8 +57,7 @@ fn stat(filename_str: &str) -> io::Result<libc::stat> {
         if rc == 0 {
             Ok(st)
         } else {
-            let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-            Err(io::Error::from_raw_os_error(errno))
+            Err(io::Error::last_os_error())
         }
     }
 }
@@ -126,8 +125,7 @@ fn read_mount_info() -> io::Result<MountList> {
         let mut mounts: *mut libc::statfs = std::ptr::null_mut();
         let n_mnt = libc::getmntinfo(&mut mounts, libc::MNT_WAIT);
         if n_mnt < 0 {
-            let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-            return Err(io::Error::from_raw_os_error(errno));
+            return Err(io::Error::last_os_error());
         }
 
         let mounts: &[libc::statfs] = std::slice::from_raw_parts(mounts as _, n_mnt as _);
@@ -150,8 +148,7 @@ fn read_mount_info() -> io::Result<MountList> {
         let mnt_mode = CString::new("r").unwrap();
         let f = libc::setmntent(path_mnt.as_ptr(), mnt_mode.as_ptr());
         if f.is_null() {
-            let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-            return Err(io::Error::from_raw_os_error(errno));
+            return Err(io::Error::last_os_error());
         }
 
         loop {
@@ -168,8 +165,7 @@ fn read_mount_info() -> io::Result<MountList> {
             let mut mount: libc::statfs = std::mem::zeroed();
             let rc = libc::statfs(dirname.as_ptr(), &mut mount);
             if rc < 0 {
-                let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-                return Err(io::Error::from_raw_os_error(errno));
+                return Err(io::Error::last_os_error());
             }
 
             info.push(&mount, devname, dirname);

diff --git a/process/src/nice.rs b/process/src/nice.rs
@@ -51,8 +51,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
 
     let res = unsafe { libc::nice(args.niceval) };
     if res < 0 {
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        let e = io::Error::from_raw_os_error(errno);
+        let e = io::Error::last_os_error();
         eprintln!("nice: {}", e);
         return Err(Box::new(e));
     }

diff --git a/process/src/renice.rs b/process/src/renice.rs
@@ -106,8 +106,7 @@ fn xsetpriority(which: u32, id: u32, prio: i32) -> io::Result<()> {
     let res = unsafe { libc::setpriority(which as i32, id, prio) };
 
     if res < 0 {
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        let e = io::Error::from_raw_os_error(errno);
+        let e = io::Error::last_os_error();
         eprintln!("setpriority: {}", e);
         Err(e)
     } else {

diff --git a/sys/src/ipcrm.rs b/sys/src/ipcrm.rs
@@ -60,8 +60,7 @@ fn msg_key_lookup(msgkey: i32) -> io::Result<i32> {
     let res: i32 = unsafe { msgget(msgkey, 0) };
 
     if res < 0 {
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        Err(io::Error::from_raw_os_error(errno))
+        Err(io::Error::last_os_error())
     } else {
         Ok(res)
     }
@@ -78,8 +77,7 @@ fn msg_rm(msgid: i32) -> io::Result<i32> {
     };
 
     if res < 0 {
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        Err(io::Error::from_raw_os_error(errno))
+        Err(io::Error::last_os_error())
     } else {
         Ok(res)
     }
@@ -92,8 +90,7 @@ fn shm_key_lookup(shmkey: i32) -> io::Result<i32> {
     let res: i32 = unsafe { shmget(shmkey, 0, 0) };
 
     if res < 0 {
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        Err(io::Error::from_raw_os_error(errno))
+        Err(io::Error::last_os_error())
     } else {
         Ok(res)
     }
@@ -109,8 +106,7 @@ fn shm_rm(shmid: i32) -> io::Result<i32> {
     };
 
     if res < 0 {
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        Err(io::Error::from_raw_os_error(errno))
+        Err(io::Error::last_os_error())
     } else {
         Ok(res)
     }
@@ -123,8 +119,7 @@ fn sem_key_lookup(semkey: i32) -> io::Result<i32> {
     let res: i32 = unsafe { semget(semkey, 0, 0) };
 
     if res < 0 {
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        Err(io::Error::from_raw_os_error(errno))
+        Err(io::Error::last_os_error())
     } else {
         Ok(res)
     }
@@ -145,8 +140,7 @@ fn sem_rm(semid: i32) -> io::Result<i32> {
     let res: i32 = unsafe { semctl(semid, 0, libc::IPC_RMID, arg) };
 
     if res < 0 {
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        Err(io::Error::from_raw_os_error(errno))
+        Err(io::Error::last_os_error())
     } else {
         Ok(res)
     }

diff --git a/users/src/mesg.rs b/users/src/mesg.rs
@@ -55,8 +55,7 @@ fn stat_tty() -> io::Result<(i32, libc::stat)> {
         let ret = libc::fstat(fd, &mut st);
         if ret < 0 {
             eprintln!("{}", gettext("fstat failed"));
-            let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-            return Err(io::Error::from_raw_os_error(errno));
+            return Err(io::Error::last_os_error());
         }
 
         Ok((fd, st))
@@ -106,8 +105,7 @@ fn set_mesg(fd: i32, st: libc::stat, setting: &str) -> io::Result<()> {
     let chres = unsafe { libc::fchmod(fd, mode) };
     if chres < 0 {
         eprintln!("{}", gettext("failed to change terminal mode"));
-        let errno = std::io::Error::last_os_error().raw_os_error().unwrap();
-        return Err(io::Error::from_raw_os_error(errno));
+        return Err(io::Error::last_os_error());
     }
 
     Ok(())