clippy lints

godot-codegen/src/class_generator.rs

@@ -915,6 +915,7 @@ fn make_virtual_methods_trait(
     let virtual_methods = locate_all_virtual_methods(class, all_bases, ctx);
     // TODO (HACK) Have to clone virtual_methods here to avoid a
     // double borrow on ctx. Can we avoid it?
+    #[allow(clippy::needless_collect)]
     let virtual_methods: Vec<_> = virtual_methods.into_iter().cloned().collect();
 
     let virtual_method_fns: Vec<_> = virtual_methods
@@ -931,6 +932,7 @@ fn make_virtual_methods_trait(
 
     quote! {
         #[allow(unused_variables)]
+        #[allow(clippy::unimplemented)]
         pub trait #trait_name: crate::private::You_forgot_the_attribute__godot_api + crate::obj::GodotClass {
             #( #virtual_method_fns )*
         }
@@ -983,15 +985,15 @@ fn locate_all_virtual_methods<'a>(
     // Get virtuals defined on the current class.
     all_virtuals.extend(
         get_methods_in_class(class)
-            .into_iter()
+            .iter()
             .filter(|meth| meth.is_virtual),
     );
     // Add virtuals from superclasses.
     for base in all_bases {
         if let Some(superclass) = ctx.get_engine_class(base) {
             all_virtuals.extend(
                 get_methods_in_class(superclass)
-                    .into_iter()
+                    .iter()
                     .filter(|meth| meth.is_virtual),
             );
         }
@@ -1010,7 +1012,7 @@ fn virtual_method_name(class_method: &ClassMethod) -> String {
     // Matching the C++ convention, we remove the leading underscore
     // from virtual method names.
     let mut method_name = class_method.name.clone();
-    if method_name.starts_with("_") {
+    if method_name.starts_with('_') {
         method_name.remove(0);
     }
 

godot-codegen/src/context.rs

@@ -52,7 +52,7 @@ impl<'a> Context<'a> {
     }
 
     pub fn get_engine_class(&self, class_name: &TyName) -> Option<&Class> {
-        self.engine_classes.get(class_name).map(|x| *x)
+        self.engine_classes.get(class_name).copied()
     }
 
     // pub fn is_engine_class(&self, class_name: &str) -> bool {