Merge pull request #97 from Tropix126/fix/smart-port-eaddrinuse

fix: handle `EADDRINUSE` in smart port errors
vexide · Mar 1, 2024 · edcb299 · edcb299
2 parents 8ff512e + d62e667
commit edcb299
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -36,6 +36,7 @@ Before releasing:
 
 - Fix error handling and error type variats in ADI bindings
 - Fix `AsynRobot` only running opcontrol
+- Properly handle `EADDRINUSE` return for smart port errors (**Breaking Change**) (#97)
 
 ### Changed
 

diff --git a/packages/pros/src/error.rs b/packages/pros/src/error.rs
@@ -97,9 +97,12 @@ pub enum PortError {
     PortOutOfRange,
     /// The specified port couldn't be configured as the specified type.
     PortCannotBeConfigured,
+    /// The specified port is already being used or is mismatched.
+    AlreadyInUse,
 }
 
 map_errno!(PortError {
     ENXIO => Self::PortOutOfRange,
     ENODEV => Self::PortCannotBeConfigured,
+    EADDRINUSE => Self::AlreadyInUse,
 });