Clean up the code a little bit

unshell-lib/src/announcement.rs

@@ -2,6 +2,7 @@ use bincode::{Decode, Encode};
 
 use crate::config::RuntimeConfig;
 
+/// Mostly temporary server message type
 #[derive(Debug, Encode, Decode)]
 pub enum Announcement {
     TestAnnouncement(String),

unshell-lib/src/client/client_runtime.rs

@@ -85,10 +85,7 @@ impl ClientRuntime {
 }
 
 impl ModuleRuntime for ClientRuntime {
-    // fn init(&mut self) {}
-
     fn is_running(&self) -> bool {
-        // println!("Checking if running");
         !self.thread_handle.is_finished()
     }
 
@@ -97,6 +94,5 @@ impl ModuleRuntime for ClientRuntime {
             self.join_signal.store(true, Ordering::Relaxed);
             let _ = self.thread_handle.join();
         }
-        // drop(self);
     }
 }

unshell-lib/src/components.rs

@@ -2,6 +2,7 @@ use unshell_obfuscate::obfuscated_symbol;
 
 use crate::config::NamedComponent;
 
+/// Publicly facing accessor function for the payload to load inside the breakout modules.
 #[obfuscated_symbol]
 pub fn get_components() -> Vec<NamedComponent> {
     return vec![

unshell-lib/src/logger/mod.rs

@@ -41,6 +41,7 @@ impl Logger for DefaultLogger {
 }
 
 pub fn set_logger_box(logger: Box<dyn Logger>) {
+    #[cfg(feature = "log")]
     unsafe {
         LOGGER = Box::leak(logger);
     }