Remove unnecessarily allowed lints

Author: taiki-e

futures-channel/tests/mpsc.rs

@@ -262,7 +262,6 @@ fn stress_shared_bounded_hard() {
     t.join().unwrap();
 }
 
-#[allow(clippy::same_item_push)]
 #[test]
 fn stress_receiver_multi_task_bounded_hard() {
     #[cfg(miri)]
@@ -411,7 +410,6 @@ async fn stress_poll_ready_sender(mut sender: mpsc::Sender<u32>, count: u32) {
 }
 
 /// Tests that after `poll_ready` indicates capacity a channel can always send without waiting.
-#[allow(clippy::same_item_push)]
 #[test]
 fn stress_poll_ready() {
     #[cfg(miri)]

futures-task/src/future_obj.rs

@@ -25,7 +25,6 @@ pub struct LocalFutureObj<'a, T> {
 impl<T> Unpin for LocalFutureObj<'_, T> {}
 
 #[allow(single_use_lifetimes)]
-#[allow(clippy::transmute_ptr_to_ptr)]
 unsafe fn remove_future_lifetime<'a, T>(
     ptr: *mut (dyn Future<Output = T> + 'a),
 ) -> *mut (dyn Future<Output = T> + 'static) {

futures-task/src/lib.rs

@@ -11,7 +11,6 @@
         allow(dead_code, unused_assignments, unused_variables)
     )
 ))]
-#![allow(clippy::non_send_fields_in_send_ty)]
 
 #[cfg(feature = "alloc")]
 extern crate alloc;

futures-task/src/waker.rs

@@ -28,7 +28,6 @@ where
 // FIXME: panics on Arc::clone / refcount changes could wreak havoc on the
 // code here. We should guard against this by aborting.
 
-#[allow(clippy::redundant_clone)] // The clone here isn't actually redundant.
 unsafe fn increase_refcount<T: ArcWake>(data: *const ()) {
     // Retain Arc, but don't touch refcount by wrapping in ManuallyDrop
     let arc = mem::ManuallyDrop::new(Arc::<T>::from_raw(data.cast::<T>()));

futures-util/src/compat/compat01as03.rs

@@ -321,7 +321,6 @@ where
 
 struct NotifyWaker(task03::Waker);
 
-#[allow(missing_debug_implementations)] // false positive: this is private type
 #[derive(Clone)]
 struct WakerToHandle<'a>(&'a task03::Waker);
 

futures-util/src/future/try_select.rs

@@ -63,7 +63,6 @@ where
     A: TryFuture,
     B: TryFuture,
 {
-    #[allow(clippy::type_complexity)]
     type Output = Result<Either<(A::Ok, B), (B::Ok, A)>, Either<(A::Error, B), (B::Error, A)>>;
 
     fn poll(mut self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<Self::Output> {

futures-util/src/lib.rs

@@ -18,7 +18,6 @@
     )
 ))]
 #![cfg_attr(docsrs, feature(doc_cfg))]
-#![allow(clippy::non_send_fields_in_send_ty)]
 #![allow(clippy::needless_borrow)] // https://github.com/rust-lang/futures-rs/pull/2558#issuecomment-1030745203
 
 #[cfg(all(feature = "bilock", not(feature = "unstable")))]

futures-util/src/stream/try_stream/try_chunks.rs

@@ -42,7 +42,6 @@ impl<St: TryStream> TryChunks<St> {
 }
 
 impl<St: TryStream> Stream for TryChunks<St> {
-    #[allow(clippy::type_complexity)]
     type Item = Result<Vec<St::Ok>, TryChunksError<St::Ok, St::Error>>;
 
     fn poll_next(mut self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<Option<Self::Item>> {

futures/tests/async_await_macros.rs

@@ -1,5 +1,3 @@
-#![allow(clippy::collapsible_match)] // https://github.com/rust-lang/rust-clippy/issues/7591
-
 use futures::channel::{mpsc, oneshot};
 use futures::executor::block_on;
 use futures::future::{self, poll_fn, FutureExt};