remove static from pointer sized or smaller types for aesthetics, change constexpr static to static constexpr for consistency

Signed-off-by: arades79 <scravers@protonmail.com>

2 files changed, 5 insertions, 5 deletions

diff --git a/src/common/hex_util.h b/src/common/hex_util.h
index 6b024588b..a00904939 100644
--- a/src/common/hex_util.h
+++ b/src/common/hex_util.h
@@ -47,7 +47,7 @@ template <typename ContiguousContainer>
     static_assert(std::is_same_v<typename ContiguousContainer::value_type, u8>,
                   "Underlying type within the contiguous container must be u8.");
 
-    constexpr static std::size_t pad_width = 2;
+    constexpr std::size_t pad_width = 2;
 
     std::string out;
     out.reserve(std::size(data) * pad_width);
diff --git a/src/common/tiny_mt.h b/src/common/tiny_mt.h
index 4689fd55b..5d5ebf158 100644
--- a/src/common/tiny_mt.h
+++ b/src/common/tiny_mt.h
@@ -223,7 +223,7 @@ public:
 
     float GenerateRandomF32() {
         // Floats have 24 bits of mantissa.
-        constexpr static u32 MantissaBits = 24;
+        constexpr u32 MantissaBits = 24;
         return static_cast<float>(GenerateRandomU24()) * (1.0f / (1U << MantissaBits));
     }
 
@@ -234,9 +234,9 @@ public:
         // Nintendo does not. They use (32 - 5) = 27 bits from the first rnd32()
         // call, and (32 - 6) bits from the second. We'll do what they do, but
         // There's not a clear reason why.
-        constexpr static u32 MantissaBits = 53;
-        constexpr static u32 Shift1st = (64 - MantissaBits) / 2;
-        constexpr static u32 Shift2nd = (64 - MantissaBits) - Shift1st;
+        constexpr u32 MantissaBits = 53;
+        constexpr u32 Shift1st = (64 - MantissaBits) / 2;
+        constexpr u32 Shift2nd = (64 - MantissaBits) - Shift1st;
 
         const u32 first = (this->GenerateRandomU32() >> Shift1st);
         const u32 second = (this->GenerateRandomU32() >> Shift2nd);