From 09b8b940256c9f5e734e4cb6eeb7117d47e81152 Mon Sep 17 00:00:00 2001 From: Dezhi Wu Date: Sat, 10 Aug 2024 10:25:36 +0800 Subject: [PATCH] Remove unnecessary `unsafe` block The `unsafe` is not needed here, as the `select64` is already a safe wrapper around the `C_select64` function. Closes #10659 --- src/fd_readable_set.rs | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/src/fd_readable_set.rs b/src/fd_readable_set.rs index c76c9d28f..28479e70f 100644 --- a/src/fd_readable_set.rs +++ b/src/fd_readable_set.rs @@ -70,23 +70,19 @@ impl FdReadableSet { let null = std::ptr::null_mut(); use crate::libc::{select64, timeval64}; if timeout_usec == Self::kNoTimeout { - unsafe { - return select64( - self.nfds_, - &mut self.fdset_, - null, - null, - std::ptr::null_mut(), - ); - } + return select64( + self.nfds_, + &mut self.fdset_, + null, + null, + std::ptr::null_mut(), + ); } else { let mut tvs = timeval64 { tv_sec: (timeout_usec / kUsecPerSec).try_into().unwrap(), tv_usec: (timeout_usec % kUsecPerSec).try_into().unwrap(), }; - unsafe { - return select64(self.nfds_, &mut self.fdset_, null, null, &mut tvs); - } + return select64(self.nfds_, &mut self.fdset_, null, null, &mut tvs); } }