# HG changeset patch
# User Rubidium <rubidium@openttd.org>
# Date 2024-02-08 20:35:33
# Node ID f692bd3cca4733f2a491d759439ed2dc46ecad6f
# Parent  123ae3cb001700d522ba3bc8c20d50dcfe7a39c3

Feature: Fully customisable number format per translation

diff --git a/src/strings.cpp b/src/strings.cpp
--- a/src/strings.cpp
+++ b/src/strings.cpp
@@ -55,6 +55,8 @@ const LanguageMetadata *_current_languag
 
 TextDirection _current_text_dir; ///< Text direction of the currently selected language.
 
+static NumberFormatSeparators _number_format_separators;
+
 #ifdef WITH_ICU_I18N
 std::unique_ptr<icu::Collator> _current_collator;    ///< Collator for the language currently in use.
 #endif /* WITH_ICU_I18N */
@@ -384,48 +386,42 @@ static const char *GetDecimalSeparator()
 	return decimal_separator;
 }
 
+void InitializeNumberFormats()
+{
+	ParseNumberFormatSeparators(_number_format_separators, _current_language->number_format);
+}
+
 /**
  * Format a number into a string.
- * @param builder   the string builder to write to
- * @param number    the number to write down
- * @param last      the last element in the buffer
- * @param separator the thousands-separator to use
+ * @param builder The string builder to write to.
+ * @param number The number to write down.
+ * @param separators The separator to use between each of the digits.
  */
-static void FormatNumber(StringBuilder &builder, int64_t number, const char *separator)
+static void FormatNumber(StringBuilder &builder, int64_t number, const NumberFormatSeparators &separators)
 {
-	static const int max_digits = 20;
-	uint64_t divisor = 10000000000000000000ULL;
-	int thousands_offset = (max_digits - 1) % 3;
-
 	if (number < 0) {
 		builder += '-';
 		number = -number;
 	}
 
+	uint64_t divisor = 10000000000000000000ULL;
 	uint64_t num = number;
 	uint64_t tot = 0;
-	for (int i = 0; i < max_digits; i++) {
+	for (size_t i = 0; i < separators.size(); i++) {
 		uint64_t quot = 0;
 		if (num >= divisor) {
 			quot = num / divisor;
 			num = num % divisor;
 		}
-		if ((tot |= quot) || i == max_digits - 1) {
+		if ((tot |= quot) != 0 || i == separators.size() - 1) {
 			builder += '0' + quot; // quot is a single digit
-			if ((i % 3) == thousands_offset && i < max_digits - 1) builder += separator;
+			builder += separators[i].data();
 		}
 
 		divisor /= 10;
 	}
 }
 
-static void FormatCommaNumber(StringBuilder &builder, int64_t number)
-{
-	const char *separator = _settings_game.locale.digit_group_separator.c_str();
-	if (StrEmpty(separator)) separator = _langpack.langpack->digit_group_separator;
-	FormatNumber(builder, number, separator);
-}
-
 static void FormatNoCommaNumber(StringBuilder &builder, int64_t number)
 {
 	fmt::format_to(builder, "{}", number);
@@ -541,10 +537,7 @@ static void FormatGenericCurrency(String
 		}
 	}
 
-	const char *separator = _settings_game.locale.digit_group_separator_currency.c_str();
-	if (StrEmpty(separator)) separator = _currency->separator.c_str();
-	if (StrEmpty(separator)) separator = _langpack.langpack->digit_group_separator_currency;
-	FormatNumber(builder, number, separator);
+	FormatNumber(builder, number, _number_format_separators);
 	if (number_str != STR_NULL) {
 		auto tmp_params = ArrayStringParameters<0>();
 		FormatString(builder, GetStringPtr(number_str), tmp_params);
@@ -1189,7 +1182,7 @@ static void FormatString(StringBuilder &
 				}
 
 				case SCC_COMMA: // {COMMA}
-					FormatCommaNumber(builder, args.GetNextParameter<int64_t>());
+					FormatNumber(builder, args.GetNextParameter<int64_t>(), _number_format_separators);
 					break;
 
 				case SCC_DECIMAL: { // {DECIMAL}
@@ -1199,7 +1192,7 @@ static void FormatString(StringBuilder &
 					int64_t divisor = PowerOfTen(digits);
 					int64_t fractional = number % divisor;
 					number /= divisor;
-					FormatCommaNumber(builder, number);
+					FormatNumber(builder, number, _number_format_separators);
 					fmt::format_to(builder, "{}{:0{}d}", GetDecimalSeparator(), fractional, digits);
 					break;
 				}
@@ -1246,7 +1239,7 @@ static void FormatString(StringBuilder &
 						}
 					}
 
-					FormatCommaNumber(builder, amount);
+					FormatNumber(builder, amount, _number_format_separators);
 					break;
 				}
 
@@ -1983,6 +1976,8 @@ bool ReadLanguagePack(const LanguageMeta
 	}
 #endif /* WITH_ICU_I18N */
 
+	InitializeNumberFormats();
+
 	Layouter::Initialize();
 
 	/* Some lists need to be sorted again after a language change. */