From 8b91bb4f0c67a2aba28e01b5aebc0fa97b8a269e Mon Sep 17 00:00:00 2001
From: meehl <12648828+meehl@users.noreply.github.com>
Date: Sat, 19 Sep 2026 19:09:07 +0200
Subject: [PATCH] Fix unintended utf8.reverse argument

Wrap gsub calls in parentheses to prevent the substitution count from
being passed as the second argument to utf8.reverse.
---
 src/Modules/Common.lua | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Modules/Common.lua b/src/Modules/Common.lua
index fb9ff0fdc8..c0e2e4259d 100644
--- a/src/Modules/Common.lua
+++ b/src/Modules/Common.lua
@@ -814,7 +814,7 @@ local function formatNumSepInner(m)
 	local x, y, minus, integer, fraction = str:find("(-?)(%d+)(%.?%d*)")
 	if main.showThousandsSeparators then
 		rev1kSep = utf8.reverse(main.thousandsSeparator)
-		integer = utf8.reverse(utf8.gsub(utf8.reverse(integer), "(%d%d%d)", "%1" .. rev1kSep))
+		integer = utf8.reverse((utf8.gsub(utf8.reverse(integer), "(%d%d%d)", "%1" .. rev1kSep)))
 		-- There will be leading separators if the number of digits are divisible by 3
 		-- This checks for their presence and removes them
 		-- Don't use patterns here because thousandsSeparator can be a pattern control character, and will crash if used
@@ -825,7 +825,7 @@ local function formatNumSepInner(m)
 			end
 		end
 	else
-		integer = utf8.reverse(utf8.gsub(utf8.reverse(integer), "(%d%d%d)", "%1"))
+		integer = utf8.reverse((utf8.gsub(utf8.reverse(integer), "(%d%d%d)", "%1")))
 	end
 	return colour .. minus .. integer .. utf8.gsub(fraction, "%.", main.decimalSeparator)
 end