max/embassy
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Removed the closing state as it was not required

Browse Source
This commit is contained in:
huntc 2021-07-14 13:31:23 +10:00
parent a247fa4f2c
commit d86892ca56
1 changed files with 13 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
embassy/src/util/mpsc.rs
View File

@ -390,7 +390,6 @@ struct ChannelState<T, const N: usize> {
read_pos: usize, read_pos: usize,
write_pos: usize, write_pos: usize,
full: bool, full: bool,
closing: bool,
closed: bool, closed: bool,
receiver_registered: bool, receiver_registered: bool,
senders_registered: u32, senders_registered: u32,
@ -407,7 +406,6 @@ impl<T, const N: usize> ChannelState<T, N> {
read_pos: 0, read_pos: 0,
write_pos: 0, write_pos: 0,
full: false, full: false,
closing: false,
closed: false, closed: false,
receiver_registered: false, receiver_registered: false,
senders_registered: 0, senders_registered: 0,
@ -528,25 +526,18 @@ where
fn try_recv_with_context(&mut self, cx: Option<&mut Context<'_>>) -> Result<T, TryRecvError> { fn try_recv_with_context(&mut self, cx: Option<&mut Context<'_>>) -> Result<T, TryRecvError> {
let mut state = &mut self.state; let mut state = &mut self.state;
self.mutex.lock(|_| { self.mutex.lock(|_| {
if !state.closed {
if state.read_pos != state.write_pos || state.full { if state.read_pos != state.write_pos || state.full {
if state.full { if state.full {
state.full = false; state.full = false;
state.senders_waker.wake(); state.senders_waker.wake();
} }
let message = let message = unsafe { (state.buf[state.read_pos]).assume_init_mut().get().read() };
unsafe { (state.buf[state.read_pos]).assume_init_mut().get().read() };
state.read_pos = (state.read_pos + 1) % state.buf.len(); state.read_pos = (state.read_pos + 1) % state.buf.len();
Ok(message) Ok(message)
} else if !state.closing { } else if !state.closed {
cx.into_iter() cx.into_iter()
.for_each(|cx| Self::set_receiver_waker(&mut state, &cx.waker())); .for_each(|cx| Self::set_receiver_waker(&mut state, &cx.waker()));
Err(TryRecvError::Empty) Err(TryRecvError::Empty)
} else {
state.closed = true;
state.senders_waker.wake();
Err(TryRecvError::Closed)
}
} else { } else {
Err(TryRecvError::Closed) Err(TryRecvError::Closed)
} }
@ -588,7 +579,7 @@ where
let state = &mut self.state; let state = &mut self.state;
self.mutex.lock(|_| { self.mutex.lock(|_| {
state.receiver_waker.wake(); state.receiver_waker.wake();
state.closing = true; state.closed = true;
}); });
} }
@ -599,7 +590,7 @@ where
fn is_closed_with_context(&mut self, cx: Option<&mut Context<'_>>) -> bool { fn is_closed_with_context(&mut self, cx: Option<&mut Context<'_>>) -> bool {
let mut state = &mut self.state; let mut state = &mut self.state;
self.mutex.lock(|_| { self.mutex.lock(|_| {
if state.closing || state.closed { if state.closed {
cx.into_iter() cx.into_iter()
.for_each(|cx| Self::set_senders_waker(&mut state, &cx.waker())); .for_each(|cx| Self::set_senders_waker(&mut state, &cx.waker()));
true true
@ -642,7 +633,7 @@ where
state.senders_registered -= 1; state.senders_registered -= 1;
if state.senders_registered == 0 { if state.senders_registered == 0 {
state.receiver_waker.wake(); state.receiver_waker.wake();
state.closing = true; state.closed = true;
} }
}) })
} }