Probably good to also name both variants FoldMembers

Signed-off-by: Christian Parpart <[email protected]>

Author: christianparpart

include/reflection-cpp/reflection.hpp

@@ -639,29 +639,6 @@ constexpr void template_for(F&& f)
     }(std::make_integer_sequence<t, E - B> {});
 }
 
-/// Folds over the members of a type
-///
-/// @param initialValue The initial value to fold with
-/// @param callable     The callable to fold with. The parameters are the member name,
-///                     the member's default value and the current result of the fold.
-///
-/// @return The result of the fold
-template <typename Object, typename Callable, typename ResultType>
-constexpr ResultType FoldType(ResultType initialValue, Callable const& callable)
-{
-    // clang-format off
-    ResultType result = initialValue;
-    template_for<0, Reflection::CountMembers<Object>>(
-        [&]<size_t I>() {
-            result = callable(Reflection::MemberNameOf<I, Object>,
-                              std::get<I>(Reflection::ToTuple(Object {})),
-                              result);
-        }
-    );
-    // clang-format on
-    return result;
-}
-
 template <auto P>
     requires(std::is_member_pointer_v<decltype(P)>)
 consteval std::string_view GetName()
@@ -719,6 +696,29 @@ void CallOnMembers(Object& object, Callable&& callable)
         [&]<auto I>() { callable(Reflection::MemberNameOf<I, Object>, std::get<I>(Reflection::ToTuple(object))); });
 }
 
+/// Folds over the members of a type without an object of it.
+///
+/// @param initialValue The initial value to fold with
+/// @param callable     The callable to fold with. The parameters are the member name,
+///                     the member's default value and the current result of the fold.
+///
+/// @return The result of the fold
+template <typename Object, typename Callable, typename ResultType>
+constexpr ResultType FoldMembers(ResultType initialValue, Callable const& callable)
+{
+    // clang-format off
+    ResultType result = initialValue;
+    template_for<0, Reflection::CountMembers<Object>>(
+        [&]<size_t I>() {
+            result = callable(Reflection::MemberNameOf<I, Object>,
+                              std::get<I>(Reflection::ToTuple(Object {})),
+                              result);
+        }
+    );
+    // clang-format on
+    return result;
+}
+
 /// Folds over the members of an object
 ///
 /// @param object       The object to fold over

test-reflection-cpp.cpp

@@ -68,10 +68,10 @@ TEST_CASE("nested", "[reflection]")
     CHECK(result == R"(a=1 b=2 c=3 d="hello" e={name="John Doe" email="[email protected]" age=42})");
 }
 
-TEST_CASE("FoldType", "[reflection]")
+TEST_CASE("FoldMembers.type", "[reflection]")
 {
     // clang-format off
-    auto const result = Reflection::FoldType<TestStruct>(0, [](auto&& /*name*/, auto&& /*value*/, auto&& result) {
+    auto const result = Reflection::FoldMembers<TestStruct>(0, [](auto&& /*name*/, auto&& /*value*/, auto&& result) {
         return result + 1;
     });
     // clang-format on
@@ -85,7 +85,7 @@ struct S
     int c {};
 };
 
-TEST_CASE("FoldMembers", "[reflection]")
+TEST_CASE("FoldMembers.value", "[reflection]")
 {
     auto const s = S { 1, 2, 3 };
     auto const result = Reflection::FoldMembers(