Fixed nits and oversight in Soc

ctru-rs/src/gfx.rs

@@ -220,6 +220,7 @@ impl From<Side> for ctru_sys::gfx3dSide_t {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::Error;
 
     #[test]
     fn gfx_duplicate() {

ctru-rs/src/lib.rs

@@ -27,10 +27,11 @@ pub fn init() {
     }
 
     #[cfg(not(test))]
-    _panic_hook_setup();
+    panic_hook_setup();
 }
 
-fn _panic_hook_setup() {
+#[cfg(not(test))]
+fn panic_hook_setup() {
     use crate::services::hid::{Hid, KeyPad};
     use std::panic::PanicInfo;
 
@@ -45,14 +46,15 @@ fn _panic_hook_setup() {
         if main_thread == std::thread::current().id() && console::Console::exists() {
             println!("\nPress SELECT to exit the software");
 
-            let hid = Hid::init().unwrap();
+            match Hid::init() {
-
+                Ok(hid) => loop {
-            loop {
+                    hid.scan_input();
-                hid.scan_input();
+                    let keys = hid.keys_down();
-                let keys = hid.keys_down();
+                    if keys.contains(KeyPad::KEY_SELECT) {
-                if keys.contains(KeyPad::KEY_SELECT) {
+                        break;
-                    break;
+                    }
-                }
+                },
+                Err(e) => println!("Error while intializing Hid controller during panic: {e}"),
             }
         }
     });

ctru-rs/src/romfs.rs

@@ -52,7 +52,7 @@ mod tests {
     use super::*;
 
     #[test]
-    fn romfs_duplicate() {
+    fn romfs_counter() {
         let _romfs = RomFS::init().unwrap();
         let value = *ROMFS_ACTIVE.lock().unwrap();
 

ctru-rs/src/services/reference.rs

@@ -2,7 +2,7 @@ use crate::Error;
 use std::sync::Mutex;
 pub(crate) struct ServiceReference {
     counter: &'static Mutex<usize>,
-    close: Box<dyn Fn()>,
+    close: Box<dyn Fn() + Send + Sync>,
 }
 
 impl ServiceReference {
@@ -14,9 +14,9 @@ impl ServiceReference {
     ) -> crate::Result<Self>
     where
         S: FnOnce() -> crate::Result<()>,
-        E: Fn() + 'static,
+        E: Fn() + Send + Sync + 'static,
     {
-        let mut value = counter.lock().unwrap(); // todo: handle poisoning
+        let mut value = counter.lock().expect("Mutex Counter for ServiceReference is poisoned"); // todo: handle poisoning
 
         if *value == 0 {
             start()?;
@@ -35,7 +35,7 @@ impl ServiceReference {
 
 impl Drop for ServiceReference {
     fn drop(&mut self) {
-        let mut value = self.counter.lock().unwrap(); // should probably handle poisoning - could just map_err to ignore it.
+        let mut value = self.counter.lock().expect("Mutex Counter for ServiceReference is poisoned"); // todo: handle poisoning
         *value -= 1;
         if *value == 0 {
             (self.close)();

ctru-rs/src/services/soc.rs

@@ -33,7 +33,7 @@ impl Soc {
     pub fn init_with_buffer_size(num_bytes: usize) -> crate::Result<Self> {
         let _service_handler = ServiceReference::new(
             &SOC_ACTIVE,
-            true,
+            false,
             || {
                 let soc_mem = unsafe { memalign(0x1000, num_bytes) } as *mut u32;
                 let r = unsafe { ctru_sys::socInit(soc_mem, num_bytes as u32) };
@@ -65,18 +65,12 @@ impl Soc {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::Error;
 
     #[test]
     fn soc_duplicate() {
         let _soc = Soc::init().unwrap();
-        let value = *SOC_ACTIVE.lock().unwrap();
 
-        assert_eq!(value, 1);
+        assert!(matches!(Soc::init(), Err(Error::ServiceAlreadyActive)))
-
-        drop(_soc);
-
-        let value = *SOC_ACTIVE.lock().unwrap();
-
-        assert_eq!(value, 0);
     }
 }