refactor(sdk): improve the log message source line reference (#2924)

# Description of Changes

Currently with the function maybe_log_error to log errors we hide the
real source line where the error happened. So to still have the same
logic but with more accurate source line, I moved it to a macro to have
the same abstraction for logging.

# API and ABI breaking changes

- 

# Expected complexity level and risk

1

# Testing

- [ ] I have tested it my application and it now show the better source
line of the log message

---------

Signed-off-by: ResuBaka <ResuBaka@users.noreply.github.com>
Co-authored-by: Phoebe Goldman <phoebe@goldman-tribe.org>

Author: ResuBaka

crates/sdk/src/websocket.rs

@@ -200,6 +200,17 @@ fn request_insert_auth_header(req: &mut http::Request<()>, token: Option<&str>)
     }
 }
 
+/// If `res` evaluates to `Err(e)`, log a warning in the form `"{}: {:?}", $cause, e`.
+///
+/// Could be trivially written as a function, but macro-ifying it preserves the source location of the log.
+macro_rules! maybe_log_error {
+    ($cause:expr, $res:expr) => {
+        if let Err(e) = $res {
+            log::warn!("{}: {:?}", $cause, e);
+        }
+    };
+}
+
 impl WsConnection {
     pub(crate) async fn connect(
         host: Uri,
@@ -241,12 +252,6 @@ impl WsConnection {
         WebSocketMessage::Binary(bsatn::to_vec(&msg).unwrap().into())
     }
 
-    fn maybe_log_error<T, U: std::fmt::Debug>(cause: &str, res: std::result::Result<T, U>) {
-        if let Err(e) = res {
-            log::warn!("{cause}: {e:?}");
-        }
-    }
-
     async fn message_loop(
         mut self,
         incoming_messages: mpsc::UnboundedSender<ServerMessage<BsatnFormat>>,
@@ -303,9 +308,9 @@ impl WsConnection {
                     },
 
                     Err(e) => {
-                        Self::maybe_log_error::<(), _>(
+                        maybe_log_error!(
                             "Error reading message from read WebSocket stream",
-                            Err(e),
+                            Result::<(), _>::Err(e)
                         );
                         break;
                     },
@@ -314,13 +319,13 @@ impl WsConnection {
                         idle = false;
                         record_metrics(bytes.len());
                         match Self::parse_response(&bytes) {
-                            Err(e) => Self::maybe_log_error::<(), _>(
+                            Err(e) => maybe_log_error!(
                                 "Error decoding WebSocketMessage::Binary payload",
-                                Err(e),
+                                Result::<(), _>::Err(e)
                             ),
-                            Ok(msg) => Self::maybe_log_error(
+                            Ok(msg) => maybe_log_error!(
                                 "Error sending decoded message to incoming_messages queue",
-                                incoming_messages.unbounded_send(msg),
+                                incoming_messages.unbounded_send(msg)
                             ),
                         }
                     }
@@ -376,7 +381,7 @@ impl WsConnection {
                         }
                     }
                     None => {
-                        Self::maybe_log_error("Error sending close frame", SinkExt::close(&mut self.sock).await);
+                        maybe_log_error!("Error sending close frame", SinkExt::close(&mut self.sock).await);
                         outgoing_messages = None;
                     }
                 },