From 8493bf389e7e783a3604b83f0f900eb6e9121306 Mon Sep 17 00:00:00 2001
From: Mateusz Pusz <mateusz.pusz@gmail.com>
Date: Tue, 18 Jan 2022 12:16:18 +0100
Subject: [PATCH] refactor: Forward declaration of `as_magnitude()` is not
 needed

---
 src/core/include/units/magnitude.h | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/src/core/include/units/magnitude.h b/src/core/include/units/magnitude.h
index 8e9c46ca..61d7e1a7 100644
--- a/src/core/include/units/magnitude.h
+++ b/src/core/include/units/magnitude.h
@@ -241,16 +241,6 @@ static constexpr bool is_magnitude<magnitude<BPs...>> = true;
 template<typename T>
 concept Magnitude = detail::is_magnitude<T>;
 
-/**
- * @brief  Convert any positive integer to a Magnitude.
- *
- * This will be the main way end users create Magnitudes.  They should rarely (if ever) create a magnitude<...> by
- * manually adding base powers.
- */
-template<ratio R>
-  requires (R.num > 0)
-constexpr Magnitude auto as_magnitude();
-
 /**
  * @brief  A base to represent pi.
  */
@@ -349,6 +339,12 @@ template<std::intmax_t N>
 static constexpr auto prime_factorization_v = prime_factorization<N>::value;
 } // namespace detail
 
+/**
+ * @brief  Convert any positive integer to a Magnitude.
+ *
+ * This will be the main way end users create Magnitudes.  They should rarely (if ever) create a magnitude<...> by
+ * manually adding base powers.
+ */
 template<ratio R>
   requires (R.num > 0)
 constexpr Magnitude auto as_magnitude() {