core: fix ranges::to for rvalue ranges

lifting-bits · Jan 8, 2025 · 6e21066 · 6e21066
1 parent 7946f97
commit 6e21066
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/core/include/gap/core/ranges.hpp b/core/include/gap/core/ranges.hpp
@@ -49,13 +49,9 @@ namespace gap::ranges
     template< typename container_t, std::ranges::input_range R >
     void move_or_copy_elements(R&& rng, container_t& container) {
         if constexpr (std::is_rvalue_reference_v< decltype(rng) >) {
-            std::move(std::ranges::begin(rng), std::ranges::end(rng),
-                std::back_inserter(container)
-            );
+            std::ranges::move(rng, std::back_inserter(container));
         } else {
-            std::copy(std::ranges::begin(rng), std::ranges::end(rng),
-                std::back_inserter(container)
-            );
+            std::ranges::copy(rng, std::back_inserter(container));
         }
     }
 
@@ -69,7 +65,7 @@ namespace gap::ranges
             result_container_t container;
 
             if constexpr (has_reserve< result_container_t >) {
-                if constexpr (requires { std::ranges::size(range); }) {
+                if constexpr (std::ranges::sized_range<R>) {
                     container.reserve(std::ranges::size(range));
                 }
             }