Skip to content

Commit 1339c87

Browse files
committed
Fix warnings and rebase artifacts
1 parent 1c67907 commit 1339c87

File tree

9 files changed

+12
-41
lines changed

9 files changed

+12
-41
lines changed

futures-channel/src/mpsc/mod.rs

-4
Original file line numberDiff line numberDiff line change
@@ -649,11 +649,7 @@ impl<T> Sender<T> {
649649
/// - `Err(SendError)` if the receiver has been dropped.
650650
pub fn poll_ready(
651651
&mut self,
652-
<<<<<<< HEAD
653652
waker: &Waker,
654-
=======
655-
waker: &Waker
656-
>>>>>>> Rename noop/panic waker and fix other compilation issue
657653
) -> Poll<Result<(), SendError>> {
658654
let inner = self.0.as_mut().ok_or(SendError {
659655
kind: SendErrorKind::Disconnected,

futures-executor/src/local_pool.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ impl LocalPool {
152152
}
153153
}
154154

155-
self.poll_pool(waker);
155+
let _ = self.poll_pool(waker);
156156
Poll::Pending
157157
})
158158
}

futures-util/src/macros/poll.rs

-19
Original file line numberDiff line numberDiff line change
@@ -14,25 +14,6 @@ macro_rules! try_ready {
1414
}
1515
}
1616

17-
18-
/// Extracts `Poll<T>` from `Poll<Result<T, E>>`.
19-
///
20-
/// This macro bakes in propagation of `Err` signals by returning early.
21-
/// This macro bakes in propagation of `Pending` and `Err` signals by returning early.
22-
#[macro_export]
23-
macro_rules! try_poll {
24-
($x:expr) => {
25-
match $x {
26-
$crate::core_reexport::task::Poll::Ready(Ok(x)) =>
27-
$crate::core_reexport::task::Poll::Ready(x),
28-
$crate::core_reexport::task::Poll::Ready(Err(e)) =>
29-
return $crate::core_reexport::task::Poll::Ready(Err(e.into())),
30-
$crate::core_reexport::task::Poll::Pending =>
31-
$crate::core_reexport::task::Poll::Pending,
32-
}
33-
}
34-
}
35-
3617
/// Extracts the successful type of a `Poll<T>`.
3718
///
3819
/// This macro bakes in propagation of `Pending` signals by returning early.

futures-util/src/sink/fanout.rs

+6-6
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@ impl<Si1, Si2> Sink for Fanout<Si1, Si2>
5454
mut self: Pin<&mut Self>,
5555
waker: &Waker,
5656
) -> Poll<Result<(), Self::SinkError>> {
57-
let sink1_ready = try_poll!(self.as_mut().sink1().poll_ready(waker)).is_ready();
58-
let sink2_ready = try_poll!(self.as_mut().sink2().poll_ready(waker)).is_ready();
57+
let sink1_ready = self.as_mut().sink1().poll_ready(waker)?.is_ready();
58+
let sink2_ready = self.as_mut().sink2().poll_ready(waker)?.is_ready();
5959
let ready = sink1_ready && sink2_ready;
6060
if ready { Poll::Ready(Ok(())) } else { Poll::Pending }
6161
}
@@ -73,8 +73,8 @@ impl<Si1, Si2> Sink for Fanout<Si1, Si2>
7373
mut self: Pin<&mut Self>,
7474
waker: &Waker,
7575
) -> Poll<Result<(), Self::SinkError>> {
76-
let sink1_ready = try_poll!(self.as_mut().sink1().poll_flush(waker)).is_ready();
77-
let sink2_ready = try_poll!(self.as_mut().sink2().poll_flush(waker)).is_ready();
76+
let sink1_ready = self.as_mut().sink1().poll_flush(waker)?.is_ready();
77+
let sink2_ready = self.as_mut().sink2().poll_flush(waker)?.is_ready();
7878
let ready = sink1_ready && sink2_ready;
7979
if ready { Poll::Ready(Ok(())) } else { Poll::Pending }
8080
}
@@ -83,8 +83,8 @@ impl<Si1, Si2> Sink for Fanout<Si1, Si2>
8383
mut self: Pin<&mut Self>,
8484
waker: &Waker,
8585
) -> Poll<Result<(), Self::SinkError>> {
86-
let sink1_ready = try_poll!(self.as_mut().sink1().poll_close(waker)).is_ready();
87-
let sink2_ready = try_poll!(self.as_mut().sink2().poll_close(waker)).is_ready();
86+
let sink1_ready = self.as_mut().sink1().poll_close(waker)?.is_ready();
87+
let sink2_ready = self.as_mut().sink2().poll_close(waker)?.is_ready();
8888
let ready = sink1_ready && sink2_ready;
8989
if ready { Poll::Ready(Ok(())) } else { Poll::Pending }
9090
}

futures-util/src/sink/with_flat_map.rs

+2-4
Original file line numberDiff line numberDiff line change
@@ -91,10 +91,8 @@ where
9191
if let Some(mut some_stream) = stream.as_mut().as_pin_mut() {
9292
while let Some(x) = ready!(some_stream.as_mut().poll_next(waker)) {
9393
let item = try_ready!(Poll::Ready(x));
94-
match try_poll!(sink.as_mut().poll_ready(waker)) {
95-
Poll::Ready(()) => {
96-
try_poll!(Poll::Ready(sink.as_mut().start_send(item)))
97-
}
94+
match sink.as_mut().poll_ready(waker)? {
95+
Poll::Ready(()) => sink.as_mut().start_send(item)?,
9896
Poll::Pending => {
9997
*buffer = Some(item);
10098
return Poll::Pending;

futures-util/src/stream/forward.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ where
5151
debug_assert!(self.buffered_item.is_none());
5252
{
5353
let mut sink = self.as_mut().sink().as_pin_mut().unwrap();
54-
if try_poll!(sink.as_mut().poll_ready(waker)).is_ready() {
54+
if sink.as_mut().poll_ready(waker)?.is_ready() {
5555
return Poll::Ready(sink.start_send(item));
5656
}
5757
}

futures-util/src/stream/select_all.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ impl<St: Stream + Unpin> Stream for SelectAll<St> {
8686
// FuturesUnordered thinks it isn't terminated
8787
// because it yielded a Some. Here we poll it
8888
// so it can realize it is terminated.
89-
self.inner.poll_next_unpin(waker);
89+
let _ = self.inner.poll_next_unpin(waker);
9090
Poll::Ready(None)
9191
}
9292
Poll::Ready(_) => Poll::Ready(None),

futures-util/src/task/noop_waker.rs

-4
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
//! Utilities for creating zero-cost wakers that don't do anything.
2-
<<<<<<< HEAD
32
use futures_core::task::{RawWaker, RawWakerVTable, Waker};
4-
=======
5-
use futures_core::task::{Waker, RawWaker, RawWakerVTable};
6-
>>>>>>> Fix remaining compilation errors
73
use core::ptr::null;
84
use core::cell::UnsafeCell;
95

futures/src/lib.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ compile_error!("The `cfg-target-has-atomic` feature requires the `nightly` featu
5353
// Macro reexports
5454
pub use futures_util::{
5555
// Error/readiness propagation
56-
try_ready, try_poll, ready,
56+
try_ready, ready,
5757
};
5858
#[cfg(feature = "std")]
5959
pub use futures_util::{

0 commit comments

Comments
 (0)