From fc8d4b5c683694c3287ef13975d2f231822287c1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?L=C3=A9o=20Lam?= <leo@leolam.fr>
Date: Sun, 13 Feb 2022 01:16:38 +0100
Subject: Fix matching issue in ksys::phys::motionTypeFromText

---
 src/KingSystem/Physics/System/physDefines.cpp | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)

(limited to 'src')

diff --git a/src/KingSystem/Physics/System/physDefines.cpp b/src/KingSystem/Physics/System/physDefines.cpp
index 2b75af66..5f505dc6 100644
--- a/src/KingSystem/Physics/System/physDefines.cpp
+++ b/src/KingSystem/Physics/System/physDefines.cpp
@@ -84,17 +84,23 @@ WallCode wallCodeFromText(const sead::SafeString& text) {
     return 0;
 }
 
-// duplicated branches?
-#ifdef NON_MATCHING
 MotionType motionTypeFromText(const sead::SafeString& text) {
-    if (text == "Dynamic")
-        return MotionType::Dynamic;
-    if (text == "Fixed")
-        return MotionType::Fixed;
-    if (text == "Keyframed")
-        return MotionType::Keyframed;
+    static constexpr const char* texts[] = {
+        "Dynamic",
+        "Fixed",
+        "Keyframed",
+    };
+    static_assert(int(MotionType::Dynamic) == 0);
+    static_assert(int(MotionType::Fixed) == 1);
+    static_assert(int(MotionType::Keyframed) == 2);
+
+    int type = 0;
+    for (const char* type_text : texts) {
+        if (text == type_text)
+            return static_cast<MotionType>(type);
+        ++type;
+    }
     return MotionType::Unknown;
 }
-#endif
 
 }  // namespace ksys::phys
-- 
cgit v1.2.3