Fix sort order for cmp(string)

It is still system-dependent, but shorter strings should come sooner. See nim-lang#8930 and nim-lang#10559
bio-nim · Sep 2, 2019 · ab4721f · ab4721f
1 parent 545ddf4
commit ab4721f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/system.nim b/lib/system.nim
@@ -3641,7 +3641,7 @@ when not defined(JS): #and not defined(nimscript):
         let minlen = min(x.len, y.len)
         result = int(nimCmpMem(x.cstring, y.cstring, minlen.csize))
         if result == 0:
-          result = x.len - y.len
+          result = y.len - x.len
 
   when declared(newSeq):
     proc cstringArrayToSeq*(a: cstringArray, len: Natural): seq[string] =