From 457bee58e2f0c9fe35fe6a02cab41456d8e59b07 Mon Sep 17 00:00:00 2001 From: Mivort Date: Sun, 16 Feb 2025 11:49:32 +0000 Subject: [PATCH] Add #[allow(...)] directives to macro-generated entries This patch adds `#[allow(non_camel_case_types)]` and `#[allow(non_upper_case_globals)]` directives to generated `const`s and `struct`s. While `cargo check` by default ignores naming style inconsistencies in macro-produced code, `rust-analyzer` seems to be using a more strict mode. So annotation of generated entries allows to prevent the display of such warnings by the language server. --- godot-macros/src/class/derive_godot_class.rs | 1 + godot-macros/src/derive/derive_from_godot.rs | 1 + 2 files changed, 2 insertions(+) diff --git a/godot-macros/src/class/derive_godot_class.rs b/godot-macros/src/class/derive_godot_class.rs index 756b48a50..b5d8de845 100644 --- a/godot-macros/src/class/derive_godot_class.rs +++ b/godot-macros/src/class/derive_godot_class.rs @@ -142,6 +142,7 @@ pub fn derive_godot_class(item: venial::Item) -> ParseResult { let funcs_collection_struct_name = format_funcs_collection_struct(class_name); let funcs_collection_struct = quote! { #[doc(hidden)] + #[allow(non_camel_case_types)] pub struct #funcs_collection_struct_name {} }; diff --git a/godot-macros/src/derive/derive_from_godot.rs b/godot-macros/src/derive/derive_from_godot.rs index 8b83efab0..28fd4ad38 100644 --- a/godot-macros/src/derive/derive_from_godot.rs +++ b/godot-macros/src/derive/derive_from_godot.rs @@ -75,6 +75,7 @@ fn make_fromgodot_for_int_enum( #( // Interesting: using let instead of const would introduce a runtime bug. Its values cannot be used in match lhs (binding). // However, bindings silently shadow variables, so the first match arm always runs; no warning in generated proc-macro code. + #[allow(non_upper_case_globals)] const #ord_variables: #int = #discriminants; )*