From 0346a4a4d32a03257a7fc2ad9d3825fe7866c278 Mon Sep 17 00:00:00 2001 From: nbresler Date: Thu, 18 Apr 2024 10:21:26 -0700 Subject: [PATCH 1/2] Prepend details namesapce --- src/core/include/mp-units/bits/external/type_name.h | 4 ++++ src/core/include/mp-units/bits/magnitude.h | 2 +- src/core/include/mp-units/quantity_spec.h | 6 +++--- src/core/include/mp-units/unit.h | 2 +- 4 files changed, 9 insertions(+), 5 deletions(-) diff --git a/src/core/include/mp-units/bits/external/type_name.h b/src/core/include/mp-units/bits/external/type_name.h index d4867033..5502cc57 100644 --- a/src/core/include/mp-units/bits/external/type_name.h +++ b/src/core/include/mp-units/bits/external/type_name.h @@ -6,6 +6,8 @@ #include #endif +namespace mp_units::detail { + template [[nodiscard]] consteval std::string_view type_name() { @@ -40,3 +42,5 @@ template else return v2; } + +} // namespace mp_units::detail \ No newline at end of file diff --git a/src/core/include/mp-units/bits/magnitude.h b/src/core/include/mp-units/bits/magnitude.h index 3b0f7648..cc9bef1d 100644 --- a/src/core/include/mp-units/bits/magnitude.h +++ b/src/core/include/mp-units/bits/magnitude.h @@ -594,7 +594,7 @@ consteval bool less(MagnitudeSpec auto lhs, MagnitudeSpec auto rhs) using rhs_base_t = decltype(get_base_value(rhs)); if constexpr (is_named_magnitude && is_named_magnitude) - return type_name() < type_name(); + return detail::type_name() < detail::type_name(); else if constexpr (!is_named_magnitude && !is_named_magnitude) return get_base_value(lhs) < get_base_value(rhs); else diff --git a/src/core/include/mp-units/quantity_spec.h b/src/core/include/mp-units/quantity_spec.h index 3679b19e..8539cf5f 100644 --- a/src/core/include/mp-units/quantity_spec.h +++ b/src/core/include/mp-units/quantity_spec.h @@ -96,7 +96,7 @@ template } template -struct quantity_spec_less : std::bool_constant() < type_name()> {}; +struct quantity_spec_less : std::bool_constant() < detail::type_name()> {}; template using type_list_of_quantity_spec_less = expr_less; @@ -639,7 +639,7 @@ template else if constexpr (rhs == dimension_one) return true; else - return type_name() < type_name(); + return detail::type_name() < detail::type_name(); } template rhs_compl) || (lhs_compl == rhs_compl && ingredients_dimension_less(Lhs::dimension, Rhs::dimension)) || (lhs_compl == rhs_compl && Lhs::dimension == Rhs::dimension && - type_name() < type_name())> {}; + detail::type_name() < detail::type_name())> {}; template using type_list_of_ingredients_less = expr_less; diff --git a/src/core/include/mp-units/unit.h b/src/core/include/mp-units/unit.h index 5c9fdf9a..abca45ac 100644 --- a/src/core/include/mp-units/unit.h +++ b/src/core/include/mp-units/unit.h @@ -644,7 +644,7 @@ template return u2; else // TODO Check if there is a better choice here - return better_type_name(u1, u2); + return detail::better_type_name(u1, u2); } else { constexpr auto canonical_lhs = get_canonical_unit(U1{}); constexpr auto canonical_rhs = get_canonical_unit(U2{}); From 7705d8b7465ef18daab08b783956f2566f4788bc Mon Sep 17 00:00:00 2001 From: nbresler Date: Thu, 18 Apr 2024 10:35:01 -0700 Subject: [PATCH 2/2] New line --- src/core/include/mp-units/bits/external/type_name.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/core/include/mp-units/bits/external/type_name.h b/src/core/include/mp-units/bits/external/type_name.h index 5502cc57..24bdc8a6 100644 --- a/src/core/include/mp-units/bits/external/type_name.h +++ b/src/core/include/mp-units/bits/external/type_name.h @@ -43,4 +43,4 @@ template return v2; } -} // namespace mp_units::detail \ No newline at end of file +} // namespace mp_units::detail