Tiny clean up to the code.

Author: treeform

src/jsony.nim

@@ -358,9 +358,14 @@ proc dumpHook*(s: var string, v: bool) =
 when defined(release):
   {.push checks: off.}
 
-const lookup = ['0', '0', '0', '1', '0', '2', '0', '3', '0', '4', '0', '5', '0', '6', '0', '7', '0', '8', '0', '9', '1', '0', '1', '1', '1', '2', '1', '3', '1', '4', '1', '5', '1', '6', '1', '7', '1', '8', '1', '9', '2', '0', '2', '1', '2', '2', '2', '3', '2', '4', '2', '5', '2', '6', '2', '7', '2', '8', '2', '9', '3', '0', '3', '1', '3', '2', '3', '3', '3', '4', '3', '5', '3', '6', '3', '7', '3', '8',
-'3', '9', '4', '0', '4', '1', '4', '2', '4', '3', '4', '4', '4', '5', '4', '6', '4', '7', '4', '8', '4', '9', '5', '0', '5', '1', '5', '2', '5', '3', '5', '4', '5', '5', '5', '6', '5', '7', '5', '8', '5', '9', '6', '0', '6', '1', '6', '2', '6', '3', '6', '4', '6', '5', '6', '6', '6', '7', '6', '8', '6', '9', '7', '0', '7', '1', '7', '2', '7', '3', '7', '4', '7', '5', '7', '6', '7', '7', '7', '8', '7', '9', '8', '0', '8', '1', '8', '2', '8', '3', '8', '4', '8', '5', '8', '6', '8', '7',
-'8', '8', '8', '9', '9', '0', '9', '1', '9', '2', '9', '3', '9', '4', '9', '5', '9', '6', '9', '7', '9', '8', '9', '9', '1', '0', '0']
+const lookup = block:
+  ## Generate 00, 01, 02 ... 99 pairs.
+  var s = ""
+  for i in 0 ..< 100:
+    if ($i).len == 1:
+      s.add("0")
+    s.add($i)
+  s
 
 template grow(s: var string, amount: int) =
   s.setLen(s.len + amount)
@@ -391,9 +396,8 @@ proc dumpHook*(s: var string, v: uint|uint8|uint16|uint32|uint64) =
       dec p
     s.grow(p)
     dec p
-    var ss = cast[ptr UncheckedArray[char]](s[0].addr)
     while p >= 0:
-      ss[at] = digits[p]
+      s[at] = digits[p]
       dec p
       inc at
 

tests/bench.nim

@@ -39,12 +39,12 @@ echo treeStr[0 ..< 100]
 echo genId, " node tree:"
 
 when not defined(gcArc):
-  timeIt "status-im/nim-json-serialization", 1000:
+  timeIt "status-im/nim-json-serialization", 100:
     keep json_serialization.Json.decode(treeStr, Node)
 
   doAssert json_serialization.Json.decode(treeStr, Node).toJson() == treeStr
 
-timeIt "treeform/jsony", 1000:
+timeIt "treeform/jsony", 100:
   keep jsony.fromJson[Node](treeStr)
 
 when defined(packedjson):