From 534627a04c77aa6791be25ad04deefa9b37034f5 Mon Sep 17 00:00:00 2001
From: Mason Reed <mason@vector35.com>
Date: Sun, 3 Aug 2025 16:59:02 -0400
Subject: [Rust] Take download callbacks by reference to avoid boxing

---
 rust/src/download_provider.rs | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

(limited to 'rust/src')

diff --git a/rust/src/download_provider.rs b/rust/src/download_provider.rs
index 9fd803a7..b0c25b42 100644
--- a/rust/src/download_provider.rs
+++ b/rust/src/download_provider.rs
@@ -134,14 +134,13 @@ impl DownloadInstance {
     pub fn perform_request(
         &mut self,
         url: &str,
-        callbacks: DownloadInstanceOutputCallbacks,
+        callbacks: &DownloadInstanceOutputCallbacks,
     ) -> Result<(), String> {
-        let callbacks = Box::into_raw(Box::new(callbacks));
         let mut cbs = BNDownloadInstanceOutputCallbacks {
             writeCallback: Some(Self::o_write_callback),
-            writeContext: callbacks as *mut c_void,
+            writeContext: callbacks as *const _ as *mut c_void,
             progressCallback: Some(Self::o_progress_callback),
-            progressContext: callbacks as *mut c_void,
+            progressContext: callbacks as *const _ as *mut c_void,
         };
 
         let url_raw = url.to_cstr();
@@ -153,8 +152,6 @@ impl DownloadInstance {
             )
         };
 
-        // Drop it
-        unsafe { drop(Box::from_raw(callbacks)) };
         if result < 0 {
             Err(self.get_error())
         } else {
@@ -206,7 +203,7 @@ impl DownloadInstance {
         method: &str,
         url: &str,
         headers: I,
-        callbacks: DownloadInstanceInputOutputCallbacks,
+        callbacks: &DownloadInstanceInputOutputCallbacks,
     ) -> Result<DownloadResponse, String>
     where
         I: IntoIterator<Item = (String, String)>,
@@ -226,14 +223,13 @@ impl DownloadInstance {
             header_value_ptrs.push(value.as_ptr());
         }
 
-        let callbacks = Box::into_raw(Box::new(callbacks));
         let mut cbs = BNDownloadInstanceInputOutputCallbacks {
             readCallback: Some(Self::i_read_callback),
-            readContext: callbacks as *mut c_void,
+            readContext: callbacks as *const _ as *mut c_void,
             writeCallback: Some(Self::i_write_callback),
-            writeContext: callbacks as *mut c_void,
+            writeContext: callbacks as *const _ as *mut c_void,
             progressCallback: Some(Self::i_progress_callback),
-            progressContext: callbacks as *mut c_void,
+            progressContext: callbacks as *const _ as *mut c_void,
         };
 
         let mut response: *mut BNDownloadInstanceResponse = null_mut();
-- 
cgit v1.3.1