squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Remove unnecessary unsafe block from condvar_atomics & mutex_atomics

Browse source
This commit is contained in:
chansuke 2020-10-24 18:22:18 +09:00
parent d147f78e36
commit d37b8cf729
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
library/std/src/sys/wasm/condvar_atomics.rs
View file

@ -53,7 +53,7 @@ impl Condvar {
#[inline] #[inline]
pub unsafe fn notify_all(&self) { pub unsafe fn notify_all(&self) {
self.cnt.fetch_add(1, SeqCst); self.cnt.fetch_add(1, SeqCst);
// SAFETY: memory_atomic_notify()is always valid // SAFETY: ptr() is always valid
unsafe { unsafe {
wasm32::memory_atomic_notify(self.ptr(), u32::MAX); // -1 == "wake everyone" wasm32::memory_atomic_notify(self.ptr(), u32::MAX); // -1 == "wake everyone"
} }

4
library/std/src/sys/wasm/mutex_atomics.rs
View file

@ -50,7 +50,7 @@ impl Mutex {
#[inline] #[inline]
pub unsafe fn try_lock(&self) -> bool { pub unsafe fn try_lock(&self) -> bool {
unsafe { self.locked.compare_exchange(0, 1, SeqCst, SeqCst).is_ok() } self.locked.compare_exchange(0, 1, SeqCst, SeqCst).is_ok()
} }
#[inline] #[inline]
@ -86,7 +86,7 @@ unsafe impl Sync for ReentrantMutex {}
impl ReentrantMutex { impl ReentrantMutex {
pub const unsafe fn uninitialized() -> ReentrantMutex { pub const unsafe fn uninitialized() -> ReentrantMutex {
unsafe { ReentrantMutex { owner: AtomicU32::new(0), recursions: UnsafeCell::new(0) } } ReentrantMutex { owner: AtomicU32::new(0), recursions: UnsafeCell::new(0) }
} }
pub unsafe fn init(&self) { pub unsafe fn init(&self) {