From 6f771e0ed7cd00113059df11180df92f2fb293ec Mon Sep 17 00:00:00 2001
From: Mateusz Pusz <mateusz.pusz@gmail.com>
Date: Thu, 3 Oct 2024 16:46:57 +0200
Subject: [PATCH] refactor: `shorten_T` removed

---
 .../include/mp-units/framework/magnitude.h    | 23 +++----------------
 1 file changed, 3 insertions(+), 20 deletions(-)

diff --git a/src/core/include/mp-units/framework/magnitude.h b/src/core/include/mp-units/framework/magnitude.h
index 898ee2b5..199e0b4d 100644
--- a/src/core/include/mp-units/framework/magnitude.h
+++ b/src/core/include/mp-units/framework/magnitude.h
@@ -145,33 +145,16 @@ template<MagnitudeSpec Element>
     return ratio{1};
 }
 
-// We do not want magnitude type to have the `l` literal after a value for a small integral number.
-// For example this modifies `magnitude<3l>` to be `magnitude<3>`
-template<auto V>
-[[nodiscard]] consteval auto shorten_T()
-{
-  if constexpr (std::integral<decltype(V)>) {
-    if constexpr (V <= std::numeric_limits<int>::max())
-      return static_cast<int>(V);
-    else
-      return static_cast<std::intmax_t>(V);
-  } else if constexpr (std::floating_point<decltype(V)>) {
-    return static_cast<long double>(V);
-  } else {
-    return V;
-  }
-}
-
 template<PowerVBase auto V, ratio R>
 [[nodiscard]] consteval auto power_v_or_T()
 {
   if constexpr (R.den == 1) {
     if constexpr (R.num == 1)
-      return shorten_T<V>();
+      return V;
     else
-      return power_v<shorten_T<V>(), R.num>{};
+      return power_v<V, R.num>{};
   } else {
-    return power_v<shorten_T<V>(), R.num, R.den>{};
+    return power_v<V, R.num, R.den>{};
   }
 }