refactor: we cannot use C++20 feature std::views::filter as it's not supported by older compilers that are supposed to "support C++20"

See https://en.cppreference.com/w/cpp/ranges/filter_view for the library reference

See https://en.cppreference.com/w/cpp/compiler_support/20 for compiler support:
The One Ranges Proposal | GCC 10 | Clang 13 (partial) 15  |	MSVC 19.29 (16.10)* | Apple Clang 14.0.3

Author: Xmader

include/PyEventLoop.hh

@@ -16,7 +16,6 @@
 #include <vector>
 #include <utility>
 #include <atomic>
-#include <ranges>
 
 struct PyEventLoop {
 public:
@@ -140,14 +139,10 @@ public:
     }
 
     /**
-     * @brief Get an iterator for the `AsyncHandle`s of all ref'ed timers
+     * @brief Get an iterator for the `AsyncHandle`s of all timers
      */
-    static inline auto getAllRefed() {
-      return std::views::filter(_timeoutIdMap,
-        [](AsyncHandle &timer) {
-          return timer.hasRef();
-        }
-      );
+    static inline auto &getAllTimers() {
+      return _timeoutIdMap;
     }
   protected:
     PyObject *_handle;

src/internalBinding/timers.cc

@@ -120,7 +120,9 @@ static bool getAllRefedTimersDebugInfo(JSContext *cx, unsigned argc, JS::Value *
   JS::CallArgs args = JS::CallArgsFromVp(argc, vp);
 
   JS::RootedVector<JS::Value> results(cx);
-  for (AsyncHandle &timer: AsyncHandle::getAllRefed()) {
+  for (AsyncHandle &timer: AsyncHandle::getAllTimers()) {
+    if (!timer.hasRef()) continue; // we only need ref'ed timers
+
     JS::Value debugInfo = jsTypeFactory(cx, timer.getDebugInfo());
     if (!results.append(debugInfo)) {
       // out of memory