Refactor: Resolve all clippy warnings in rs_loader (#687)

* Refactor: Resolve all clippy warnings in rs_loader

* Build: Add Clippy as a pre-build CMake target for CI

* Update CMakeLists.txt

---------

Co-authored-by: Vicente Eduardo Ferrer Garcia <7854099+viferga@users.noreply.github.com>

Author: niko091

source/loaders/rs_loader/rust/compiler/src/api/class.rs

@@ -92,13 +92,12 @@ extern "C" fn class_singleton_static_invoke(
             .expect("Unable to get method name");
         let ret = class.call(name, args);
         std::mem::forget(class);
-        std::mem::forget(name);
         ret
     };
     if let Ok(ret) = ret {
-        return ret;
+        ret
     } else {
-        return 0 as OpaqueType;
+        0 as OpaqueType
     }
 }
 
@@ -116,12 +115,10 @@ extern "C" fn class_singleton_static_await(
 
 #[no_mangle]
 extern "C" fn class_singleton_destroy(_klass: OpaqueType, class_impl: OpaqueType) {
-    if !rs_loader_destroyed() {
-        if !class_impl.is_null() {
-            unsafe {
-                let class = Box::from_raw(class_impl as *mut class::Class);
-                drop(class);
-            }
+    if !rs_loader_destroyed() && !class_impl.is_null() {
+        unsafe {
+            let class = Box::from_raw(class_impl as *mut class::Class);
+            drop(class);
         }
     }
 }
@@ -239,10 +236,8 @@ pub fn register_class(class_registration: ClassRegistration) {
                     loader_impl_type(class_registration.loader_impl, ret.as_ptr()),
                 );
             };
-        }
-        else {
-            let ret = CString::new("Null")
-            .expect("Failed to convert return type to C string");
+        } else {
+            let ret = CString::new("Null").expect("Failed to convert return type to C string");
 
             unsafe {
                 signature_set_return(
@@ -294,10 +289,8 @@ pub fn register_class(class_registration: ClassRegistration) {
                     loader_impl_type(class_registration.loader_impl, ret.as_ptr()),
                 );
             };
-        }
-        else {
-            let ret = CString::new("Null")
-            .expect("Failed to convert return type to C string");
+        } else {
+            let ret = CString::new("Null").expect("Failed to convert return type to C string");
 
             unsafe {
                 signature_set_return(

source/loaders/rs_loader/rust/compiler/src/api/function.rs

@@ -54,15 +54,12 @@ extern "C" fn function_singleton_await(
     0 as OpaqueType
 }
 
-
 #[no_mangle]
 extern "C" fn function_singleton_destroy(_func: OpaqueType, func_impl: OpaqueType) {
-    if !rs_loader_destroyed() {
-        if !func_impl.is_null() {
-            unsafe {
-                let func_ptr = Box::from_raw(func_impl as *mut class::Function);
-                drop(func_ptr);
-            }
+    if !rs_loader_destroyed() && !func_impl.is_null() {
+        unsafe {
+            let func_ptr = Box::from_raw(func_impl as *mut class::Function);
+            drop(func_ptr);
         }
     }
 }

source/loaders/rs_loader/rust/compiler/src/api/mod.rs

@@ -121,10 +121,7 @@ extern "C" {
 }
 
 pub fn get_loader_lifecycle_state(loader_impl: OpaqueType) -> *mut LoaderLifecycleState {
-    let loader_lifecycle_state =
-        unsafe { loader_impl_get(loader_impl) } as *mut LoaderLifecycleState;
-
-    loader_lifecycle_state
+    (unsafe { loader_impl_get(loader_impl) }) as *mut LoaderLifecycleState
 }
 
 static mut RS_LOADER_PTR: *mut c_void = std::ptr::null_mut();
@@ -142,7 +139,9 @@ pub fn loader_lifecycle_register(loader_impl: OpaqueType) {
 }
 
 pub fn loader_lifecycle_unload_children(loader_impl: OpaqueType) {
-    unsafe { loader_unload_children(loader_impl); }
+    unsafe {
+        loader_unload_children(loader_impl);
+    }
 }
 
 pub enum PrimitiveMetacallProtocolTypes {
@@ -182,7 +181,5 @@ pub fn define_type(
 }
 
 pub fn rs_loader_destroyed() -> bool {
-    unsafe {
-        loader_is_destroyed(RS_LOADER_PTR) == 0
-    }
+    unsafe { loader_is_destroyed(RS_LOADER_PTR) == 0 }
 }