Numbers bigger than `1L << 53` loose lower bits when parsing by default parser using Scala.js #2311

plokhotnyuk · 2024-09-17T09:23:48Z

Script to reproduce:

//> using dep "io.circe::circe-core::0.14.10"
//> using dep "io.circe::circe-parser::0.14.10"
////> using dep "io.circe::circe-jawn::0.14.10"

import io.circe.Json
import io.circe.parser._
//import io.circe.jawn._

println(parse("[9007199254740993,9999999999999999999]").getOrElse(null))

Store it to script.js and use the following command to build with latest scala-cli:

scala-cli --power package --js --js-mode release script.sc --force -o script.js

Result of running with node:

[
  9007199254740992,
  10000000000000000000
]

Expected output:

[
  9007199254740993,
  9999999999999999999
]

Works fine with circe-jawn instead of circe-parser, or using JVMs or Scala Native.

The text was updated successfully, but these errors were encountered:

hamnis · 2024-09-17T15:42:17Z

this is apparently using pure javascript JSON.parse, then converting after, so I guess standard javascript semantics should be expected.

plokhotnyuk · 2024-09-17T15:54:38Z

this is apparently using pure javascript JSON.parse, then converting after, so I guess standard javascript semantics should be expected.

Then why println(Json.arr(Json.fromBigInt(BigInt("9223372036854775808")),Json.fromBigInt(BigInt("9999999999999999999"))).noSpaces) produces [9223372036854775808,9999999999999999999] without wrapping numbers by quotation marks?

Is it expected that the library cannot parse own output without loosing information when used with Scala.js?

armanbilge · 2024-09-17T19:13:31Z

This problem is one of the reasons why I opened this issue.

Consider switching to the Jawn parser on JS #1941

My change was accepted, but only for the next major version.

Use the Jawn parser on JS #2042

Personally, I think we should backport this change as a bugfix, see #1941 (comment).

plokhotnyuk mentioned this issue Sep 20, 2024

More fair comparison of throughput of round-trip MateuszKubuszok/derivation-benchmarks#1

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Numbers bigger than `1L << 53` loose lower bits when parsing by default parser using Scala.js #2311

Numbers bigger than `1L << 53` loose lower bits when parsing by default parser using Scala.js #2311

Numbers bigger than 1L << 53 loose lower bits when parsing by default parser using Scala.js #2311

Numbers bigger than 1L << 53 loose lower bits when parsing by default parser using Scala.js #2311

Comments

Numbers bigger than `1L << 53` loose lower bits when parsing by default parser using Scala.js #2311

Numbers bigger than `1L << 53` loose lower bits when parsing by default parser using Scala.js #2311