Skip to content

Commit bdfebff

Browse files
taiki-ecramertj
authored andcommitted
Remove redundant code from TryStreamExt::{and_then, or_else} (#1570)
1 parent 31948bd commit bdfebff

File tree

2 files changed

+6
-22
lines changed

2 files changed

+6
-22
lines changed

futures-util/src/try_stream/and_then.rs

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -82,17 +82,9 @@ impl<St, Fut, F> Stream for AndThen<St, Fut, F>
8282
self.as_mut().future().set(Some(fut));
8383
}
8484

85-
assert!(self.future.is_some());
86-
match ready!(self.as_mut().future().as_pin_mut().unwrap().try_poll(cx)) {
87-
Ok(e) => {
88-
self.as_mut().future().set(None);
89-
Poll::Ready(Some(Ok(e)))
90-
}
91-
Err(e) => {
92-
self.as_mut().future().set(None);
93-
Poll::Ready(Some(Err(e)))
94-
}
95-
}
85+
let e = ready!(self.as_mut().future().as_pin_mut().unwrap().try_poll(cx));
86+
self.as_mut().future().set(None);
87+
Poll::Ready(Some(e))
9688
}
9789
}
9890

futures-util/src/try_stream/or_else.rs

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -83,17 +83,9 @@ impl<St, Fut, F> Stream for OrElse<St, Fut, F>
8383
self.as_mut().future().set(Some(fut));
8484
}
8585

86-
assert!(self.future.is_some());
87-
match ready!(self.as_mut().future().as_pin_mut().unwrap().try_poll(cx)) {
88-
Ok(e) => {
89-
self.as_mut().future().set(None);
90-
Poll::Ready(Some(Ok(e)))
91-
}
92-
Err(e) => {
93-
self.as_mut().future().set(None);
94-
Poll::Ready(Some(Err(e)))
95-
}
96-
}
86+
let e = ready!(self.as_mut().future().as_pin_mut().unwrap().try_poll(cx));
87+
self.as_mut().future().set(None);
88+
Poll::Ready(Some(e))
9789
}
9890
}
9991

0 commit comments

Comments
 (0)