Trace errors in drop

Author: ddoktorski

src/connection.rs

@@ -11,6 +11,7 @@ use dap::base_message::Sendable;
 use dap::errors::ServerError;
 use dap::prelude::{Event, Request, ResponseBody, Server};
 use dap::server::{ServerReader, ServerWriter};
+use tracing::error;
 
 pub struct Connection {
     inbound_rx: mpsc::Receiver<Request>,
@@ -107,7 +108,7 @@ fn spawn_reader_thread(
     thread::spawn(move || {
         while let Ok(Some(request)) = server_reader.poll_request() {
             if inbound_tx.send(request).is_err() {
-                // TODO: Add error tracing
+                error!("Inbound channel closed");
                 break;
             }
         }

src/debugger.rs

@@ -2,7 +2,7 @@ use anyhow::{Result, bail};
 use cairo_vm::vm::vm_core::VirtualMachine;
 use dap::events::ExitedEventBody;
 use dap::prelude::Event::{Exited, Terminated};
-use tracing::debug;
+use tracing::{debug, error};
 
 use crate::connection::Connection;
 use crate::debugger::handler::{HandleResult, NextAction};
@@ -64,9 +64,13 @@ impl CairoDebugger {
 
 impl Drop for CairoDebugger {
     fn drop(&mut self) {
-        // TODO: Add error tracing
-        // TODO: Send correct exit code
-        self.connection.send_event(Terminated(None)).ok();
-        self.connection.send_event(Exited(ExitedEventBody { exit_code: 0 })).ok();
+        if let Err(err) = self.connection.send_event(Terminated(None)) {
+            error!("Sending terminated event failed: {}", err);
+        }
+
+        // TODO(#34): Send correct exit code
+        if let Err(err) = self.connection.send_event(Exited(ExitedEventBody { exit_code: 0 })) {
+            error!("Sending exit event failed: {}", err);
+        }
     }
 }