Cleanup macro (ava-labs#598)

x/programs/rust/sdk_macros/src/lib.rs

@@ -59,10 +59,7 @@ pub fn public(_: TokenStream, item: TokenStream) -> TokenStream {
     });
 
     // Collect all parameter names and types into separate vectors.
-    let param_names: Vec<_> = full_params.clone().map(|(name, _)| name).collect();
-    // Copy the iterator so we can use it again.
-    let param_names_cloned: Vec<_> = param_names.clone();
-    let param_types: Vec<_> = full_params.map(|(_, ty)| ty).collect();
+    let (param_names, param_types): (Vec<_>, Vec<_>) = full_params.unzip();
 
     // Extract the original function's return type. This must be a WASM supported type.
     let return_type = &input.sig.output;
@@ -72,9 +69,10 @@ pub fn public(_: TokenStream, item: TokenStream) -> TokenStream {
         #[no_mangle]
         pub extern "C" fn #new_name(#(#param_names: #param_types), *) #return_type {
             // .into() uses the From() on each argument in the iterator to convert it to the type we want. 70% sure about this statement.
-            #name(#(#param_names_cloned.into()),*) // This means that every parameter type must implement From<i64>(except for the supported primitive types).
+            #name(#(#param_names.into()),*) // This means that every parameter type must implement From<i64>(except for the supported primitive types).
         }
     };
+
     TokenStream::from(output)
 }