Fix "throw error instead of panicking"

laptou · Oct 15, 2020 · d31b28e · d31b28e
1 parent 70117aa
commit d31b28e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/interface/response.rs b/src/interface/response.rs
@@ -219,7 +219,7 @@ impl Response {
                 0x0026 => Event::PhyConfigChanged {
                     selected_phys: BitFlags::from_bits_truncate(buf.get_u32_le()),
                 },
-                _ => todo!("throw error instead of panicking"),
+                _ => return Err(Error::UnknownEventCode { evt_code }),
             },
         })
     }

diff --git a/src/result.rs b/src/result.rs
@@ -22,6 +22,8 @@ pub enum Error {
     UnknownOpcode { opcode: u16 },
     #[error("Unknown command status: {:x}.", status)]
     UnknownStatus { status: u8 },
+    #[error("Unknown event code: {:x}.", evt_code)]
+    UnknownEventCode { evt_code: u16 },
     #[error("Timed out.")]
     TimedOut,
     #[error("The socket received invalid data.")]