correct clippy and stuff

crates/macros/src/function.rs

@@ -375,10 +375,10 @@ impl<'a> Args<'a> {
                         syn::Pat::Ident(syn::PatIdent { ident, .. }) => ident,
                         _ => bail!(pat => "Unsupported argument."),
                     };
-                    let nullable = type_is_nullable(&**ty)?;
+                    let nullable = type_is_nullable(ty)?;
                     result.typed.push(TypedArg {
                         name: ident,
-                        ty: &**ty,
+                        ty,
                         nullable,
                         default: defaults.remove(ident),
                     });

crates/macros/src/impl_.rs

@@ -38,7 +38,7 @@ pub struct ImplArgs {
 }
 
 /// Different types of rename rules for methods.
-#[derive(Debug, Copy, Clone, FromMeta)]
+#[derive(Debug, Default, Copy, Clone, FromMeta)]
 pub enum RenameRule {
     /// Methods won't be renamed.
     #[darling(rename = "none")]

examples/hello_world.rs

@@ -1,15 +1,4 @@
-use ext_php_rs::{
-    args::Arg,
-    builders::{ClassBuilder, FunctionBuilder, ModuleStartup},
-    class::{ClassMetadata, ConstructorMeta, ConstructorResult, RegisteredClass},
-    constant::IntoConst,
-    convert::IntoZval,
-    flags::DataType,
-    internal::class::{PhpClassImpl, PhpClassImplCollector},
-    prelude::*,
-    types::{ZendClassObject, Zval},
-    zend::ExecuteData,
-};
+use ext_php_rs::{constant::IntoConst, prelude::*, types::ZendClassObject};
 
 #[derive(Debug)]
 #[php_class]
@@ -75,12 +64,12 @@ pub struct TestZvalConvert<'a> {
 }
 
 #[php_function]
-pub fn get_zval_convert<'a>(z: TestZvalConvert<'a>) -> i32 {
+pub fn get_zval_convert(z: TestZvalConvert) -> i32 {
     dbg!(z);
     5
 }
 
-fn startup(ty: i32, mod_num: i32) -> i32 {
+fn startup(_ty: i32, mod_num: i32) -> i32 {
     5.register_constant("SOME_CONST", mod_num).unwrap();
     0
 }

src/builders/module.rs

@@ -4,7 +4,6 @@ use crate::{
     constant::IntoConst,
     error::Result,
     ffi::{ext_php_rs_php_build_id, ZEND_MODULE_API_NO},
-    flags::MethodFlags,
     zend::{FunctionEntry, ModuleEntry},
     PHP_DEBUG, PHP_ZTS,
 };

src/internal/mod.rs

@@ -12,6 +12,7 @@ pub type ModuleStartupMutex = Mutex<Option<ModuleStartup>>;
 
 /// The initialisation value for [`ModuleStartupMutex`]. By default the mutex
 /// contains [`None`].
+#[allow(clippy::declare_interior_mutable_const)]
 pub const MODULE_STARTUP_INIT: ModuleStartupMutex = const_mutex(None);
 
 /// Called by startup functions registered with the [`#[php_startup]`] macro.

tests/module.rs

@@ -39,6 +39,6 @@ pub fn hello_world(name: String) -> String {
 }
 
 #[php_module]
-pub fn module(module: ModuleBuilder) -> ModuleBuilder {
-    module
+pub fn get_module(module: ModuleBuilder) -> ModuleBuilder {
+    module.function(wrap_function!(hello_world))
 }