Skip to content

Commit

Permalink
Auto merge of #34731 - GGist:fix_sync_try_recv, r=alexcrichton
Browse files Browse the repository at this point in the history
Check for data in Receiver::try_recv before reporting disconnect

Fixes #34711

r? @alexcrichton
  • Loading branch information
bors committed Jul 10, 2016
2 parents c73bb9b + 2af340c commit 46e7f4b
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
9 changes: 9 additions & 0 deletions src/libstd/sync/mpsc/mod.rs
Expand Up @@ -2180,6 +2180,15 @@ mod sync_tests {
assert!(rx.recv().is_err());
}

#[test]
fn oneshot_single_thread_try_recv_closed_with_data() {
let (tx, rx) = sync_channel::<i32>(1);
tx.send(10).unwrap();
drop(tx);
assert_eq!(rx.try_recv(), Ok(10));
assert_eq!(rx.try_recv(), Err(TryRecvError::Disconnected));
}

#[test]
fn oneshot_single_thread_peek_data() {
let (tx, rx) = sync_channel::<i32>(1);
Expand Down
2 changes: 1 addition & 1 deletion src/libstd/sync/mpsc/sync.rs
Expand Up @@ -309,7 +309,7 @@ impl<T> Packet<T> {
let mut guard = self.lock.lock().unwrap();

// Easy cases first
if guard.disconnected { return Err(Disconnected) }
if guard.disconnected && guard.buf.size() == 0 { return Err(Disconnected) }
if guard.buf.size() == 0 { return Err(Empty) }

// Be sure to wake up neighbors
Expand Down

0 comments on commit 46e7f4b

Please sign in to comment.