Minor cleanup

taiki-e · Sep 21, 2020 · 9579d1e · 9579d1e
1 parent 99c34e0
commit 9579d1e
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 28 deletions.
diff --git a/src/lib.rs b/src/lib.rs
@@ -190,9 +190,7 @@ macro_rules! __pin_project_internal {
         [$proj_vis:vis]
         [$(#[$attrs:meta])* $vis:vis struct $ident:ident]
         [$($def_generics:tt)*]
-        [$($impl_generics:tt)*]
-        [$($ty_generics:tt)*]
-        [$(where $($where_clause:tt)*)?]
+        [$($impl_generics:tt)*] [$($ty_generics:tt)*] [$(where $($where_clause:tt)*)?]
         {
             $(
                 $(#[$pin:ident])?
@@ -216,9 +214,7 @@ macro_rules! __pin_project_internal {
             $crate::__pin_project_internal! { @make_proj_ty_struct;
                 [$proj_vis]
                 [$vis struct $ident]
-                [$($impl_generics)*]
-                [$($ty_generics)*]
-                [$(where $($where_clause)*)?]
+                [$($impl_generics)*] [$($ty_generics)*] [$(where $($where_clause)*)?]
                 {
                     $(
                         $(#[$pin])?
@@ -233,7 +229,7 @@ macro_rules! __pin_project_internal {
             {
                 $proj_vis fn project<'__pin>(
                     self: $crate::__private::Pin<&'__pin mut Self>,
-                ) -> Projection<'__pin, $($ty_generics)*> {
+                ) -> Projection <'__pin, $($ty_generics)*> {
                     unsafe {
                         let Self { $($field),* } = self.get_unchecked_mut();
                         Projection {
@@ -247,7 +243,7 @@ macro_rules! __pin_project_internal {
                 }
                 $proj_vis fn project_ref<'__pin>(
                     self: $crate::__private::Pin<&'__pin Self>,
-                ) -> ProjectionRef<'__pin, $($ty_generics)*> {
+                ) -> ProjectionRef <'__pin, $($ty_generics)*> {
                     unsafe {
                         let Self { $($field),* } = self.get_ref();
                         ProjectionRef {
@@ -263,9 +259,7 @@ macro_rules! __pin_project_internal {
 
             $crate::__pin_project_internal! { @make_unpin_impl;
                 [$vis $ident]
-                [$($impl_generics)*]
-                [$($ty_generics)*]
-                [$(where $($where_clause)*)?]
+                [$($impl_generics)*] [$($ty_generics)*] [$(where $($where_clause)*)?]
                 $(
                     $field: $crate::__pin_project_internal!(@make_unpin_bound;
                         $(#[$pin])? $field_ty
@@ -275,9 +269,7 @@ macro_rules! __pin_project_internal {
 
             $crate::__pin_project_internal! { @make_drop_impl;
                 [$ident]
-                [$($impl_generics)*]
-                [$($ty_generics)*]
-                [$(where $($where_clause)*)?]
+                [$($impl_generics)*] [$($ty_generics)*] [$(where $($where_clause)*)?]
             }
 
             // Ensure that it's impossible to use pin projections on a #[repr(packed)] struct.
@@ -299,7 +291,7 @@ macro_rules! __pin_project_internal {
                 $($where_clause)*)?
             {
                 $(
-                    &this.$field;
+                    let _ = &this.$field;
                 )+
             }
         };
@@ -310,9 +302,7 @@ macro_rules! __pin_project_internal {
     (@make_proj_ty_struct;
         [$proj_vis:vis]
         [$vis:vis struct $ident:ident]
-        [$($impl_generics:tt)*]
-        [$($ty_generics:tt)*]
-        [$(where $($where_clause:tt)* )?]
+        [$($impl_generics:tt)*] [$($ty_generics:tt)*] [$(where $($where_clause:tt)* )?]
         {
             $(
                 $(#[$pin:ident])?
@@ -353,9 +343,7 @@ macro_rules! __pin_project_internal {
     // make_unpin_impl
     (@make_unpin_impl;
         [$vis:vis $ident:ident]
-        [$($impl_generics:tt)*]
-        [$($ty_generics:tt)*]
-        [$(where $($where_clause:tt)* )?]
+        [$($impl_generics:tt)*] [$($ty_generics:tt)*] [$(where $($where_clause:tt)* )?]
         $($field:tt)*
     ) => {
         // Automatically create the appropriate conditional `Unpin` implementation.
@@ -402,9 +390,7 @@ macro_rules! __pin_project_internal {
     // make_drop_impl
     (@make_drop_impl;
         [$ident:ident]
-        [$($impl_generics:tt)*]
-        [$($ty_generics:tt)*]
-        [$(where $($where_clause:tt)* )?]
+        [$($impl_generics:tt)*] [$($ty_generics:tt)*] [$(where $($where_clause:tt)* )?]
     ) => {
         // Ensure that struct does not implement `Drop`.
         //

diff --git a/tests/expand/tests/expand/default-struct.expanded.rs b/tests/expand/tests/expand/default-struct.expanded.rs
@@ -64,8 +64,8 @@ const _: () = {
     impl<T, U> MustNotImplDrop for Struct<T, U> {}
     #[deny(safe_packed_borrows)]
     fn __assert_not_repr_packed<T, U>(this: &Struct<T, U>) {
-        &this.pinned;
-        &this.unpinned;
+        let _ = &this.pinned;
+        let _ = &this.unpinned;
     }
 };
 fn main() {}
diff --git a/tests/expand/tests/expand/pub-struct.expanded.rs b/tests/expand/tests/expand/pub-struct.expanded.rs
@@ -64,8 +64,8 @@ const _: () = {
     impl<T, U> MustNotImplDrop for Struct<T, U> {}
     #[deny(safe_packed_borrows)]
     fn __assert_not_repr_packed<T, U>(this: &Struct<T, U>) {
-        &this.pinned;
-        &this.unpinned;
+        let _ = &this.pinned;
+        let _ = &this.unpinned;
     }
 };
 fn main() {}