RichEnum::value_of() - return std::optional<V>

... instead of std::optional<std::reference_wrapper<const V>
teslamotors · Sep 16, 2024 · 5f3ba4a · 5f3ba4a
1 parent bd09021
commit 5f3ba4a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 16 deletions.
diff --git a/include/fixed_containers/enum_utils.hpp b/include/fixed_containers/enum_utils.hpp
@@ -167,8 +167,7 @@ struct RichEnumAdapter
 };
 
 template <class RichEnum>
-constexpr std::optional<std::reference_wrapper<const RichEnum>> value_of(
-    const std::string_view& name)
+constexpr std::optional<RichEnum> value_of(const std::string_view& name)
 {
     for (const RichEnum& rich_enum_val : RichEnum::values())
     {
@@ -182,8 +181,7 @@ constexpr std::optional<std::reference_wrapper<const RichEnum>> value_of(
 }
 
 template <class RichEnum>
-constexpr std::optional<std::reference_wrapper<const RichEnum>> value_of(
-    const typename RichEnum::BackingEnum& backing_enum)
+constexpr std::optional<RichEnum> value_of(const typename RichEnum::BackingEnum& backing_enum)
 {
     const auto& rich_enum_values = RichEnum::values();
 
@@ -215,7 +213,7 @@ constexpr std::optional<std::reference_wrapper<const RichEnum>> value_of(
 }
 
 template <class RichEnum>
-constexpr std::optional<std::reference_wrapper<const RichEnum>> value_of(
+constexpr std::optional<RichEnum> value_of(
     std::underlying_type_t<typename RichEnum::BackingEnum> enum_integer)
 {
     return value_of<RichEnum>(typename RichEnum::BackingEnum(enum_integer));
@@ -420,19 +418,17 @@ class SkeletalRichEnumLite
     using ValuesFriend = SkeletalRichEnumValues<RichEnumType>;
 
 public:
-    static constexpr std::optional<std::reference_wrapper<const RichEnumType>> value_of(
-        const std::string_view& name)
+    static constexpr std::optional<RichEnumType> value_of(const std::string_view& name)
     {
         return rich_enums_detail::value_of<RichEnumType>(name);
     }
 
-    static constexpr std::optional<std::reference_wrapper<const RichEnumType>> value_of(
-        const BackingEnum& backing_enum)
+    static constexpr std::optional<RichEnumType> value_of(const BackingEnum& backing_enum)
     {
         return rich_enums_detail::value_of<RichEnumType>(backing_enum);
     }
 
-    static constexpr std::optional<std::reference_wrapper<const RichEnumType>> value_of(
+    static constexpr std::optional<RichEnumType> value_of(
         std::underlying_type_t<BackingEnum> enum_integer)
     {
         return rich_enums_detail::value_of<RichEnumType>(enum_integer);

diff --git a/test/enum_utils_test.cpp b/test/enum_utils_test.cpp
@@ -237,8 +237,7 @@ TEST(RichEnum, ValueOfName)
     }
 
     {
-        constexpr const TestRichEnum1& MY_VALUE = TestRichEnum1::value_of("C_ONE").value();
-
+        constexpr TestRichEnum1 MY_VALUE = TestRichEnum1::value_of("C_ONE").value();
         static_assert(MY_VALUE == TestRichEnum1::C_ONE());
     }
 }
@@ -256,8 +255,7 @@ TEST(RichEnum, ValueOfBackingEnum)
 
     {
         using BE = detail::TestRichEnum1BackingEnum;
-        constexpr const TestRichEnum1& MY_VALUE = TestRichEnum1::value_of(BE::C_ONE).value();
-
+        constexpr TestRichEnum1 MY_VALUE = TestRichEnum1::value_of(BE::C_ONE).value();
         static_assert(MY_VALUE == TestRichEnum1::C_ONE());
     }
 }
@@ -273,8 +271,7 @@ TEST(RichEnum, ValueOfUnderlyingInt)
     }
 
     {
-        constexpr const TestRichEnum1& MY_VALUE = TestRichEnum1::value_of(19).value();
-
+        constexpr TestRichEnum1 MY_VALUE = TestRichEnum1::value_of(19).value();
         static_assert(MY_VALUE == TestRichEnum1::C_ONE());
     }
 }