From 54489a57b2ed9e9742c137c8260bf3a1b98f1bd3 Mon Sep 17 00:00:00 2001 From: Mateusz Pusz Date: Wed, 11 May 2022 22:56:58 +0200 Subject: [PATCH] refactor: missing constraints on a return type added Implementation verification and interface documentation purposes --- src/core/include/units/magnitude.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/core/include/units/magnitude.h b/src/core/include/units/magnitude.h index 4b95c6e4..bf729945 100644 --- a/src/core/include/units/magnitude.h +++ b/src/core/include/units/magnitude.h @@ -443,13 +443,13 @@ constexpr auto cbrt(magnitude m) // Magnitude product implementation. // Base cases, for when either (or both) inputs are the identity. -constexpr auto operator*(magnitude<>, magnitude<>) { return magnitude<>{}; } -constexpr auto operator*(magnitude<>, Magnitude auto m) { return m; } -constexpr auto operator*(Magnitude auto m, magnitude<>) { return m; } +constexpr Magnitude auto operator*(magnitude<>, magnitude<>) { return magnitude<>{}; } +constexpr Magnitude auto operator*(magnitude<>, Magnitude auto m) { return m; } +constexpr Magnitude auto operator*(Magnitude auto m, magnitude<>) { return m; } // Recursive case for the product of any two non-identity Magnitudes. template -constexpr auto operator*(magnitude, magnitude) +constexpr Magnitude auto operator*(magnitude, magnitude) { // Case for when H1 has the smaller base. if constexpr (H1.get_base() < H2.get_base()) {