From 91c1d2e6c62d1aa41abe5c60938655beaac06544 Mon Sep 17 00:00:00 2001
From: Cooldude2606 <25043174+Cooldude2606@users.noreply.github.com>
Date: Mon, 23 Dec 2024 15:31:04 +0000
Subject: [PATCH] Fix version variables in compat.lua (#713)

---
 packages/host/modules/clusterio/compat.lua | 23 +++++++++++-----------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/packages/host/modules/clusterio/compat.lua b/packages/host/modules/clusterio/compat.lua
index 68117d8e..96cd444a 100644
--- a/packages/host/modules/clusterio/compat.lua
+++ b/packages/host/modules/clusterio/compat.lua
@@ -96,16 +96,17 @@ function compat.version_le(version)
 	return true
 end
 
---- The versions of factorio we support
-local v2_0  = compat.version_ge("2.0")
-local v1_1  = not v2_0  and compat.version_ge("1.1")
-local v1_0  = not v1_1  and compat.version_ge("1.0")
-local v0_18 = not v1_0  and compat.version_ge("0.18")
-local v0_17 = not v0_18 and compat.version_ge("0.17")
-
+--- The major versions of factorio we support
 local v2 = compat.version_ge("2.0")
-local v1 = not v2 and compat.version_ge("1.0")
-local v0 = not v1 and compat.version_ge("0.17")
+local v1 = compat.version_ge("1.0")  and not compat.version_ge("2.0")
+local v0 = compat.version_ge("0.17") and not compat.version_ge("1.0")
+
+--- The minor versions of factorio we support
+local v2_0  = compat.version_ge("2.0")
+local v1_1  = compat.version_ge("1.1")  and not compat.version_ge("2.0")
+local v1_0  = compat.version_ge("1.0")  and not compat.version_ge("1.1")
+local v0_18 = compat.version_ge("0.18") and not compat.version_ge("1.0")
+local v0_17 = compat.version_ge("0.17") and not compat.version_ge("0.18")
 
 --- Raise an error because the current version is unsupported
 local function unsupported_version()
@@ -145,10 +146,8 @@ end
 function runtime_properties.prototypes()
 	if v0 or v1 then
 		--- Lazy loading of prototype data
+		--- Iteration is UB, we can fix this once someone actually needs it
 		return setmetatable({}, {
-			-- Prevent iteration due to lazy loading, its a really rare use case
-			__pairs = function() error("Pairs not supported in " .. current_version_str, 2) end,
-			__ipairs = function() error("IPairs not supported in " .. current_version_str, 2) end,
 			__index = function(self, key)
 				local value = game[key .. "_prototypes"]
 				rawset(self, key, value)