From c0c3892064a775b13fd5cae00f58b43bee062003 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Edgars=20C=C4=ABrulis?= <edgarsscirulis@gmail.com>
Date: Sun, 15 Jan 2023 09:47:31 +0200
Subject: [PATCH] Version.cpp: Improve version parsing to handle mixed numeric
 and alphabetic characters
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Edgars Cīrulis <edgarsscirulis@gmail.com>
---
 launcher/Version.cpp | 38 +++++++++++++++++++++++++++++++-------
 1 file changed, 31 insertions(+), 7 deletions(-)

diff --git a/launcher/Version.cpp b/launcher/Version.cpp
index b9090e29..5d814a25 100644
--- a/launcher/Version.cpp
+++ b/launcher/Version.cpp
@@ -74,12 +74,36 @@ bool Version::operator!=(const Version &other) const
 void Version::parse()
 {
     m_sections.clear();
-
-    // FIXME: this is bad. versions can contain a lot more separators...
-    QStringList parts = m_string.split('.');
-
-    for (const auto& part : parts)
-    {
-        m_sections.append(Section(part));
+    QString currentSection;
+    bool lastCharWasDigit = false;
+    for (int i = 0; i < m_string.size(); ++i) {
+        if(m_string[i].isDigit()){
+            if(!lastCharWasDigit){
+                if(!currentSection.isEmpty()){
+                    m_sections.append(Section(currentSection));
+                }
+                currentSection = "";
+            }
+            currentSection += m_string[i];
+            lastCharWasDigit = true;
+        }else if(m_string[i].isLetter()){
+            if(lastCharWasDigit){
+                if(!currentSection.isEmpty()){
+                    m_sections.append(Section(currentSection));
+                }
+                currentSection = "";
+            }
+            currentSection += m_string[i];
+            lastCharWasDigit = false;
+        }
+        else if(m_string[i] == '-' || m_string[i] == '_'){
+            if(!currentSection.isEmpty()){
+                m_sections.append(Section(currentSection));
+            }
+            currentSection = "";
+        }
+    }
+    if (!currentSection.isEmpty()) {
+        m_sections.append(Section(currentSection));
     }
 }