Ignore clippy::extra_unused_type_parameters lint (#2707)

Author: taiki-e

futures-test/src/assert_unmoved.rs

@@ -175,6 +175,8 @@ impl<T> PinnedDrop for AssertUnmoved<T> {
 
 #[cfg(test)]
 mod tests {
+    #![allow(clippy::extra_unused_type_parameters)] // https://github.com/rust-lang/rust-clippy/issues/10319
+
     use futures_core::future::Future;
     use futures_core::task::{Context, Poll};
     use futures_util::future::pending;
@@ -183,6 +185,7 @@ mod tests {
 
     use super::AssertUnmoved;
 
+    // TODO: move this to futures/tests/auto_traits.rs
     #[test]
     fn assert_send_sync() {
         fn assert<T: Send + Sync>() {}

futures/tests/auto_traits.rs

@@ -1,4 +1,5 @@
 #![cfg(feature = "compat")]
+#![allow(clippy::extra_unused_type_parameters)] // https://github.com/rust-lang/rust-clippy/issues/10318
 
 //! Assert Send/Sync/Unpin for all public types.
 

futures/tests/object_safety.rs

@@ -1,49 +1,51 @@
-fn assert_is_object_safe<T>() {}
+#![allow(clippy::extra_unused_type_parameters)] // https://github.com/rust-lang/rust-clippy/issues/10319
+
+fn assert_is_object_safe<T: ?Sized>() {}
 
 #[test]
 fn future() {
     // `FutureExt`, `TryFutureExt` and `UnsafeFutureObj` are not object safe.
     use futures::future::{FusedFuture, Future, TryFuture};
 
-    assert_is_object_safe::<&dyn Future<Output = ()>>();
-    assert_is_object_safe::<&dyn FusedFuture<Output = ()>>();
-    assert_is_object_safe::<&dyn TryFuture<Ok = (), Error = (), Output = Result<(), ()>>>();
+    assert_is_object_safe::<dyn Future<Output = ()>>();
+    assert_is_object_safe::<dyn FusedFuture<Output = ()>>();
+    assert_is_object_safe::<dyn TryFuture<Ok = (), Error = (), Output = Result<(), ()>>>();
 }
 
 #[test]
 fn stream() {
     // `StreamExt` and `TryStreamExt` are not object safe.
     use futures::stream::{FusedStream, Stream, TryStream};
 
-    assert_is_object_safe::<&dyn Stream<Item = ()>>();
-    assert_is_object_safe::<&dyn FusedStream<Item = ()>>();
-    assert_is_object_safe::<&dyn TryStream<Ok = (), Error = (), Item = Result<(), ()>>>();
+    assert_is_object_safe::<dyn Stream<Item = ()>>();
+    assert_is_object_safe::<dyn FusedStream<Item = ()>>();
+    assert_is_object_safe::<dyn TryStream<Ok = (), Error = (), Item = Result<(), ()>>>();
 }
 
 #[test]
 fn sink() {
     // `SinkExt` is not object safe.
     use futures::sink::Sink;
 
-    assert_is_object_safe::<&dyn Sink<(), Error = ()>>();
+    assert_is_object_safe::<dyn Sink<(), Error = ()>>();
 }
 
 #[test]
 fn io() {
     // `AsyncReadExt`, `AsyncWriteExt`, `AsyncSeekExt` and `AsyncBufReadExt` are not object safe.
     use futures::io::{AsyncBufRead, AsyncRead, AsyncSeek, AsyncWrite};
 
-    assert_is_object_safe::<&dyn AsyncRead>();
-    assert_is_object_safe::<&dyn AsyncWrite>();
-    assert_is_object_safe::<&dyn AsyncSeek>();
-    assert_is_object_safe::<&dyn AsyncBufRead>();
+    assert_is_object_safe::<dyn AsyncRead>();
+    assert_is_object_safe::<dyn AsyncWrite>();
+    assert_is_object_safe::<dyn AsyncSeek>();
+    assert_is_object_safe::<dyn AsyncBufRead>();
 }
 
 #[test]
 fn task() {
     // `ArcWake`, `SpawnExt` and `LocalSpawnExt` are not object safe.
     use futures::task::{LocalSpawn, Spawn};
 
-    assert_is_object_safe::<&dyn Spawn>();
-    assert_is_object_safe::<&dyn LocalSpawn>();
+    assert_is_object_safe::<dyn Spawn>();
+    assert_is_object_safe::<dyn LocalSpawn>();
 }