added missing return and default case to Button::addListener

LemLib · May 27, 2024 · f4d01f3 · f4d01f3
1 parent 9ad720f
commit f4d01f3
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/gamepad/controller.cpp b/src/gamepad/controller.cpp
@@ -17,9 +17,13 @@ uint32_t Button::onRelease(std::function<void(void)> func) const {
 
 uint32_t Button::addListener(EventType event, std::function<void(void)> func) const {
     switch (event) {
-        case Gamepad::EventType::ON_PRESS: this->onPress(std::move(func));
-        case Gamepad::EventType::ON_LONG_PRESS: this->onLongPress(std::move(func));
-        case Gamepad::EventType::ON_RELEASE: this->onRelease(std::move(func));
+        case Gamepad::EventType::ON_PRESS: return this->onPress(std::move(func));
+        case Gamepad::EventType::ON_LONG_PRESS: return this->onLongPress(std::move(func));
+        case Gamepad::EventType::ON_RELEASE: return this->onRelease(std::move(func));
+        default:
+        TODO("add error logging")
+        errno = EINVAL;
+        return 0;
     }
 }