fix: remove unecessary if, return if cancelled in cancel_timeout

Author: victor-Lopez25

Inc/HALAL/Services/Time/Scheduler.hpp

@@ -51,15 +51,15 @@ struct Scheduler {
         if(microseconds == 0) [[unlikely]] microseconds = 1;
         return register_task(microseconds, func, false);
     }
-    static inline void cancel_timeout(uint8_t id) {
+    static inline bool cancel_timeout(uint8_t id) {
         /* NOTE: This does not fix this case:
           1. id = set_timeout(x, func)
           2. timeout ends, func gets called and removed internally
           3. id_2 = set_timeout(y, func_2) // id will be equal to id_2
           4. clear_timeout(id) -> will remove the second timeout
          */
-        if(tasks_[id].repeating) return;
-        unregister_task(id);
+        if(tasks_[id].repeating) return false;
+        return unregister_task(id);
     }
 
     // static void global_timer_callback();

Src/HALAL/Services/Time/Scheduler.cpp

@@ -173,8 +173,6 @@ inline uint8_t Scheduler::allocate_slot() {
 }
 
 inline void Scheduler::release_slot(uint8_t id) {
-    // NOTE: This condition shouldn't be here since it's an internal function but it could be an assert
-    if(id >= kMaxTasks) [[unlikely]] return;
     ready_bitmap_ &= ~(1u << id);
     free_bitmap_ |= (1u << id);
 }