Skip to content

Commit

Permalink
remove unnecessary todos and includes
Browse files Browse the repository at this point in the history
  • Loading branch information
ion098 committed Jun 7, 2024
1 parent b4742e5 commit 1832d76
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
1 change: 0 additions & 1 deletion include/gamepad/controller.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@ class Controller {
* @param joystick Which joystick axis's value to return
*/
float operator[](pros::controller_analog_e_t joystick);
TODO("hide memebrs and expose getters/const refs")
const Button& L1 {m_L1};
const Button& L2 {m_L2};
const Button& R1 {m_R1};
Expand Down
3 changes: 0 additions & 3 deletions include/gamepad/event_handler.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,7 @@
#include <map>
#include <atomic>

#include "gamepad/todo.hpp"
#include "gamepad/recursive_mutex.hpp"
#include "pros/rtos.hpp"

namespace Gamepad {

Expand All @@ -35,7 +33,6 @@ template <typename... Args> class EventHandler {
bool remove_listener(uint32_t id) {
std::lock_guard lock(mutex);
if (listeners.find(id) == listeners.end()) {
TODO("change handling maybe?")
return false;
}
listeners.erase(id);
Expand Down

0 comments on commit 1832d76

Please sign in to comment.