Timer service cleans up properly

Author: vinniefalco

src/corosio/src/detail/posix/resolver_service.cpp

@@ -389,7 +389,7 @@ class posix_resolver_impl
 
     void release() override;
 
-    std::coroutine_handle<> resolve(
+    void resolve(
         std::coroutine_handle<>,
         capy::executor_ref,
         std::string_view host,
@@ -399,7 +399,7 @@ class posix_resolver_impl
         std::error_code*,
         resolver_results*) override;
 
-    std::coroutine_handle<> reverse_resolve(
+    void reverse_resolve(
         std::coroutine_handle<>,
         capy::executor_ref,
         endpoint const& ep,

src/corosio/src/detail/posix/signals.cpp

@@ -189,7 +189,7 @@ class posix_signal_impl
 
     void release() override;
 
-    std::coroutine_handle<> wait(
+    void wait(
         std::coroutine_handle<>,
         capy::executor_ref,
         std::stop_token,

src/corosio/src/detail/timer_service.cpp

@@ -107,8 +107,20 @@ class timer_service_impl : public timer_service
 
     void shutdown() override
     {
+        // Cancel all waiting timers and destroy coroutine handles
+        // This properly decrements outstanding_work_ for each waiting timer
         while (auto* impl = timers_.pop_front())
+        {
+            if (impl->waiting_)
+            {
+                impl->waiting_ = false;
+                // Destroy the coroutine handle without resuming
+                impl->h_.destroy();
+                // Decrement work count to avoid leak
+                sched_->on_work_finished();
+            }
             delete impl;
+        }
         while (auto* impl = free_list_.pop_front())
             delete impl;
     }