Fixes an issue where importing the strutils module, or any other im…

…porting the `strutils` module, ends up with a compile time error on platforms where ints are less then 32-bit wide.
nim-lang · Dec 28, 2023 · ee5c6a7 · ee5c6a7
1 parent 1324d2e
commit ee5c6a7
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/lib/pure/unicode.nim b/lib/pure/unicode.nim
@@ -846,7 +846,12 @@ proc cmpRunesIgnoreCase*(a, b: openArray[char]): int {.rtl, extern: "nuc$1".} =
     # slow path:
     fastRuneAt(a, i, ar)
     fastRuneAt(b, j, br)
-    result = RuneImpl(toLower(ar)) - RuneImpl(toLower(br))
+    when sizeof(int) < 4:
+      const lo = low(int).int32
+      const hi = high(int).int32
+      result = clamp(RuneImpl(toLower(ar)) - RuneImpl(toLower(br)), lo, hi).int
+    else:
+      result = RuneImpl(toLower(ar)) - RuneImpl(toLower(br))
     if result != 0: return
   result = a.len - b.len