From af02db61e49daa94a922c644d1b1ade86c5122ab Mon Sep 17 00:00:00 2001 From: Daniil Bondarev Date: Sat, 19 Sep 2020 23:43:30 -0700 Subject: [PATCH] Revert "Allow unused_unsafe warning" This reverts commit 234b8628456f3626f623a1f94965c1f5479a6f81. --- src/sys/quota.rs | 1 - src/sys/wait.rs | 9 --------- 2 files changed, 10 deletions(-) diff --git a/src/sys/quota.rs b/src/sys/quota.rs index caf520b9df..1933013219 100644 --- a/src/sys/quota.rs +++ b/src/sys/quota.rs @@ -23,7 +23,6 @@ struct QuotaCmd(QuotaSubCmd, QuotaType); impl QuotaCmd { #[allow(unused_unsafe)] fn as_int(&self) -> c_int { - #[allow(unused_unsafe)] unsafe { libc::QCMD(self.0 as i32, self.1 as i32) } } } diff --git a/src/sys/wait.rs b/src/sys/wait.rs index 94294c7f91..0c040427cc 100644 --- a/src/sys/wait.rs +++ b/src/sys/wait.rs @@ -119,43 +119,36 @@ impl WaitStatus { #[allow(unused_unsafe)] fn exited(status: i32) -> bool { - #[allow(unused_unsafe)] unsafe { libc::WIFEXITED(status) } } #[allow(unused_unsafe)] fn exit_status(status: i32) -> i32 { - #[allow(unused_unsafe)] unsafe { libc::WEXITSTATUS(status) } } #[allow(unused_unsafe)] fn signaled(status: i32) -> bool { - #[allow(unused_unsafe)] unsafe { libc::WIFSIGNALED(status) } } #[allow(unused_unsafe)] fn term_signal(status: i32) -> Result { - #[allow(unused_unsafe)] Signal::try_from(unsafe { libc::WTERMSIG(status) }) } #[allow(unused_unsafe)] fn dumped_core(status: i32) -> bool { - #[allow(unused_unsafe)] unsafe { libc::WCOREDUMP(status) } } #[allow(unused_unsafe)] fn stopped(status: i32) -> bool { - #[allow(unused_unsafe)] unsafe { libc::WIFSTOPPED(status) } } #[allow(unused_unsafe)] fn stop_signal(status: i32) -> Result { - #[allow(unused_unsafe)] Signal::try_from(unsafe { libc::WSTOPSIG(status) }) } @@ -166,7 +159,6 @@ fn syscall_stop(status: i32) -> bool { // of delivering SIGTRAP | 0x80 as the signal number for syscall // stops. This allows easily distinguishing syscall stops from // genuine SIGTRAP signals. - #[allow(unused_unsafe)] unsafe { libc::WSTOPSIG(status) == libc::SIGTRAP | 0x80 } } @@ -177,7 +169,6 @@ fn stop_additional(status: i32) -> c_int { #[allow(unused_unsafe)] fn continued(status: i32) -> bool { - #[allow(unused_unsafe)] unsafe { libc::WIFCONTINUED(status) } }