✨ feat(ports): Implemented scanPorts no file or directory exception

NestorDP · NestorDP · commit f1c279aaf4cf · 2025-10-19T22:00:13.000-03:00
diff --git a/include/libserial/ports.hpp b/include/libserial/ports.hpp
@@ -50,9 +50,7 @@ Ports() = default;
  * @brief Scans the system for available serial ports
  *
  * @return uint16_t The number of serial ports found
- * @throws SerialException if scanning fails
  * @throws SerialException if no ports are found
- * @throws PermissionDeniedException if insufficient permissions
  */
 uint16_t scanPorts();
 
diff --git a/src/ports.cpp b/src/ports.cpp
@@ -26,8 +26,7 @@ uint16_t Ports::scanPorts() {
   const char* by_id_dir = kSysSerialByIdPath;
   DIR* dir = opendir(by_id_dir);
   if (!dir) {
-    std::cout << "No serial devices directory: " << by_id_dir << "\n";
-    return -1;
+    throw SerialException("Error while reading " + std::string(by_id_dir) + ": " + strerror(errno));
   }
 
   // The POSIX directory-entry structure used by readdir() to describe files

Original file line number	Diff line number	Diff line change
`@@ -50,9 +50,7 @@ Ports() = default;`
`50`	`50`	`* @brief Scans the system for available serial ports`
`51`	`51`	`*`
`52`	`52`	`* @return uint16_t The number of serial ports found`
`53`		`- * @throws SerialException if scanning fails`
`54`	`53`	`* @throws SerialException if no ports are found`
`55`		`- * @throws PermissionDeniedException if insufficient permissions`
`56`	`54`	`*/`
`57`	`55`	`uint16_t scanPorts();`
`58`	`56`
Original file line number	Diff line number	Diff line change
`@@ -26,8 +26,7 @@ uint16_t Ports::scanPorts() {`
`26`	`26`	`const char* by_id_dir = kSysSerialByIdPath;`
`27`	`27`	`DIR* dir = opendir(by_id_dir);`
`28`	`28`	`if (!dir) {`
`29`		`- std::cout << "No serial devices directory: " << by_id_dir << "\n";`
`30`		`- return -1;`
	`29`	`+ throw SerialException("Error while reading " + std::string(by_id_dir) + ": " + strerror(errno));`
`31`	`30`	`}`
`32`	`31`
`33`	`32`	`// The POSIX directory-entry structure used by readdir() to describe files`