refactor: streamline memory allocation in wallet_build_and_sign_transaction

This commit simplifies the memory allocation process for the transaction bytes in the wallet_build_and_sign_transaction function by removing an unnecessary match statement. The updated code directly allocates and converts the boxed slice into a raw pointer, improving readability and maintainability.

Author: PastaPastaPasta

key-wallet-ffi/src/transaction.rs

@@ -421,9 +421,8 @@ pub unsafe extern "C" fn wallet_build_and_sign_transaction(
         let size = serialized.len();
 
         // Allocate memory for the result
-        let tx_bytes = match Vec::<u8>::with_capacity(size).into_boxed_slice() {
-            bytes => Box::into_raw(bytes) as *mut u8,
-        };
+        let bytes = Vec::<u8>::with_capacity(size).into_boxed_slice();
+        let tx_bytes = Box::into_raw(bytes) as *mut u8;
 
         // Copy the serialized transaction
         ptr::copy_nonoverlapping(serialized.as_ptr(), tx_bytes, size);