internal: suppress non_snake_case lint in #[pin_data]

Allows `non_snake_case` lint on struct fields generated by `#[pin_data]`.

Since the same warning will be reported by the compiler on the struct
definition, having extra warnings for the generated code is unnecessary
and confusing.

Reported-by: Gary Guo <gary@garyguo.net>
Link: #125
Closes: https://lore.kernel.org/rust-for-linux/DGTBJBIVFZ2K.2F1ZEFGY0G7NK@garyguo.net/
Fixes: db96c51 ("add references to previously initialized fields")
Signed-off-by: Mirko Adzic <adzicmirko97@gmail.com>

Author: Mirko-A

CHANGELOG.md

@@ -39,6 +39,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Corrected `T: Sized` bounds to `T: ?Sized` in the generated `PinnedDrop`
   check by `#[pin_data]`.
+- `#[pin_data]` no longer produces additional `non_snake_case` warnings if field names
+  are not of snake case. Standard field definition warnings are unaffected.
 
 ## [0.0.10] - 2025-08-19
 

internal/src/pin_data.rs

@@ -164,7 +164,10 @@ fn generate_unpin_impl(
     quote! {
         // This struct will be used for the unpin analysis. It is needed, because only structurally
         // pinned fields are relevant whether the struct should implement `Unpin`.
-        #[allow(dead_code)] // The fields below are never used.
+        #[allow(
+            dead_code, // The fields below are never used.
+            non_snake_case // The warning will be emitted on the struct definition.
+        )]
         struct __Unpin #generics_with_pin_lt
         #where_token
             #predicates
@@ -300,7 +303,9 @@ fn generate_projections(
     let docs = format!(" Pin-projections of [`{ident}`]");
     quote! {
         #[doc = #docs]
-        #[allow(dead_code)]
+        // Allow `non_snake_case` since the same warning will be emitted on
+        // the struct definition.
+        #[allow(dead_code, non_snake_case)]
         #[doc(hidden)]
         #vis struct #projection #generics_with_pin_lt
             #whr
@@ -393,6 +398,9 @@ fn generate_the_pin_data(
             ///   to deallocate.
             #pin_safety
             #(#attrs)*
+            // Allow `non_snake_case` since the same warning will be emitted on
+            // the struct definition.
+            #[allow(non_snake_case)]
             #vis unsafe fn #ident<E>(
                 self,
                 slot: *mut #ty,

tests/ui/expand/many_generics.expanded.rs

@@ -13,7 +13,7 @@ where
     _pin: PhantomPinned,
 }
 /// Pin-projections of [`Foo`]
-#[allow(dead_code)]
+#[allow(dead_code, non_snake_case)]
 #[doc(hidden)]
 struct FooProjection<'__pin, 'a, 'b: 'a, T: Bar<'b> + ?Sized + 'a, const SIZE: usize = 0>
 where
@@ -89,6 +89,7 @@ const _: () = {
         /// - `slot` is a valid pointer to uninitialized memory.
         /// - the caller does not touch `slot` when `Err` is returned, they are only permitted
         ///   to deallocate.
+        #[allow(non_snake_case)]
         unsafe fn array<E>(
             self,
             slot: *mut [u8; 1024 * 1024],
@@ -110,6 +111,7 @@ const _: () = {
         /// - `slot` is a valid pointer to uninitialized memory.
         /// - the caller does not touch `slot` when `Err` is returned, they are only permitted
         ///   to deallocate.
+        #[allow(non_snake_case)]
         unsafe fn r<E>(
             self,
             slot: *mut &'b mut [&'a mut T; SIZE],
@@ -132,6 +134,7 @@ const _: () = {
         /// - the caller does not touch `slot` when `Err` is returned, they are only permitted
         ///   to deallocate.
         /// - `slot` will not move until it is dropped, i.e. it will be pinned.
+        #[allow(non_snake_case)]
         unsafe fn _pin<E>(
             self,
             slot: *mut PhantomPinned,
@@ -176,7 +179,7 @@ const _: () = {
     {
         type Datee = Foo<'a, 'b, T, SIZE>;
     }
-    #[allow(dead_code)]
+    #[allow(dead_code, non_snake_case)]
     struct __Unpin<'__pin, 'a, 'b: 'a, T: Bar<'b> + ?Sized + 'a, const SIZE: usize = 0>
     where
         T: Bar<'a, 1>,

tests/ui/expand/pin-data.expanded.rs

@@ -5,7 +5,7 @@ struct Foo {
     _pin: PhantomPinned,
 }
 /// Pin-projections of [`Foo`]
-#[allow(dead_code)]
+#[allow(dead_code, non_snake_case)]
 #[doc(hidden)]
 struct FooProjection<'__pin> {
     array: &'__pin mut [u8; 1024 * 1024],
@@ -51,6 +51,7 @@ const _: () = {
         /// - `slot` is a valid pointer to uninitialized memory.
         /// - the caller does not touch `slot` when `Err` is returned, they are only permitted
         ///   to deallocate.
+        #[allow(non_snake_case)]
         unsafe fn array<E>(
             self,
             slot: *mut [u8; 1024 * 1024],
@@ -73,6 +74,7 @@ const _: () = {
         /// - the caller does not touch `slot` when `Err` is returned, they are only permitted
         ///   to deallocate.
         /// - `slot` will not move until it is dropped, i.e. it will be pinned.
+        #[allow(non_snake_case)]
         unsafe fn _pin<E>(
             self,
             slot: *mut PhantomPinned,
@@ -101,7 +103,7 @@ const _: () = {
     unsafe impl ::pin_init::__internal::PinData for __ThePinData {
         type Datee = Foo;
     }
-    #[allow(dead_code)]
+    #[allow(dead_code, non_snake_case)]
     struct __Unpin<'__pin> {
         __phantom_pin: ::core::marker::PhantomData<fn(&'__pin ()) -> &'__pin ()>,
         __phantom: ::core::marker::PhantomData<fn(Foo) -> Foo>,

tests/ui/expand/pinned_drop.expanded.rs

@@ -5,7 +5,7 @@ struct Foo {
     _pin: PhantomPinned,
 }
 /// Pin-projections of [`Foo`]
-#[allow(dead_code)]
+#[allow(dead_code, non_snake_case)]
 #[doc(hidden)]
 struct FooProjection<'__pin> {
     array: &'__pin mut [u8; 1024 * 1024],
@@ -51,6 +51,7 @@ const _: () = {
         /// - `slot` is a valid pointer to uninitialized memory.
         /// - the caller does not touch `slot` when `Err` is returned, they are only permitted
         ///   to deallocate.
+        #[allow(non_snake_case)]
         unsafe fn array<E>(
             self,
             slot: *mut [u8; 1024 * 1024],
@@ -73,6 +74,7 @@ const _: () = {
         /// - the caller does not touch `slot` when `Err` is returned, they are only permitted
         ///   to deallocate.
         /// - `slot` will not move until it is dropped, i.e. it will be pinned.
+        #[allow(non_snake_case)]
         unsafe fn _pin<E>(
             self,
             slot: *mut PhantomPinned,
@@ -101,7 +103,7 @@ const _: () = {
     unsafe impl ::pin_init::__internal::PinData for __ThePinData {
         type Datee = Foo;
     }
-    #[allow(dead_code)]
+    #[allow(dead_code, non_snake_case)]
     struct __Unpin<'__pin> {
         __phantom_pin: ::core::marker::PhantomData<fn(&'__pin ()) -> &'__pin ()>,
         __phantom: ::core::marker::PhantomData<fn(Foo) -> Foo>,