proper version sorting (#38) - not written with AI

Author: Ravbug

source/globals.cpp

@@ -7,6 +7,21 @@
 #elif _WIN32
 #include <ShlObj_core.h>
 #endif
+#include <cstdio>
+
+ParsedVersion parseVersion(const std::string_view version) {
+    int major, minor, patch, build;
+    if (sscanf(version.data(), "%d.%d.%d%*c%d", &major, &minor, &patch, &build) == 4) {
+        return {
+            major,
+            minor,
+            patch,
+            build
+        };
+    };
+    // parse error, assume all 0s.
+    return {};
+}
 
 void launch_process(const std::string& command, int flags) {
 #if defined __APPLE__ || defined __linux__
@@ -94,3 +109,4 @@ void dpi_scale(wxWindow* window){
 	//set the minimum size
 	window->SetSizeHints(wxSize(minw, minh));
 }
+

source/globals.h

@@ -139,3 +139,9 @@ struct project {
 		return this->path == other.path;
 	}
 };
+
+struct ParsedVersion {
+    int major = 0, minor = 0, patch = 0, build = 0;
+};
+
+ParsedVersion parseVersion(const std::string_view version);

source/interface_derived.cpp

@@ -741,9 +741,24 @@ int wxCALLBACK MainFrameDerived::CompareItems(wxIntPtr item1, wxIntPtr item2, wx
         case 0: // Name
             result = p1.name.compare(p2.name);
             break;
-        case 1: // Version
-            result = p1.version.compare(p2.version);
+        case 1: { // Version
+            auto v1 = parseVersion(p1.version);
+            auto v2 = parseVersion(p2.version);
+            
+            auto v1t = std::tie(v1.major, v1.minor, v1.patch, v1.build);
+            auto v2t = std::tie(v2.major, v2.minor, v2.patch, v2.build);
+            
+            if (v1t < v2t) {
+                result = -1;
+            }
+            else if (v1t > v2t) {
+                result = 1;
+            }
+            else {
+                result = 0;
+            }
             break;
+        }
         case 2: // Last Modified
             // Compare dates - note: string comparison may not work correctly for all date formats
             // For dates, default to newest first (reverse the comparison)