make a roundtrip to have stable numbers

planetis-m · Apr 18, 2024 · 5455440 · 5455440
1 parent 6834009
commit 5455440
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/jsonpak/parser.nim b/src/jsonpak/parser.nim
@@ -1,4 +1,4 @@
-import std/[parsejson, streams], private/[jsontree, jsonnode]
+import std/[parsejson, streams, strutils], private/[jsontree, jsonnode]
 export JsonParsingError
 
 proc parseJsonAtom(tree: var JsonTree; p: var JsonParser) =
@@ -10,7 +10,7 @@ proc parseJsonAtom(tree: var JsonTree; p: var JsonParser) =
     storeAtom(tree, opcodeInt, p.a)
     discard getTok(p)
   of tkFloat:
-    storeAtom(tree, opcodeFloat, p.a)
+    storeAtom(tree, opcodeFloat, $parseFloat(p.a)) # roundtrip
     discard getTok(p)
   of tkTrue:
     storeAtom(tree, opcodeTrue)

diff --git a/src/jsonpak/private/rawops_sorted.nim b/src/jsonpak/private/rawops_sorted.nim
@@ -94,8 +94,6 @@ proc rawDeduplicate*(tree: var JsonTree, n: NodePos, parents: var seq[PatchPos])
 
 proc rawHash*(tree: JsonTree, n: NodePos): Hash =
   privateAccess(JsonTree)
-  # iff we knew that BiTable.vals doesn't contain
-  # stale entries we could significantly speed this up.
   var h = Hash(0)
   let L = span(tree, n.int)
   for i in 0..<L: