jawn

jawn copied to clipboard

Parser.parseNum() no support `Long` type number?

1

@non when I use jawn lib, some error happens below: > [info] java.lang.NumberFormatException: For input string: "1493016373269" [info] at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65) [info] at java.lang.Integer.parseInt(Integer.java:583) [info] at java.lang.Integer.parseInt(Integer.java:615) [info] at jawn.support.json4s.Parser$$anon$1.jint(Parser.scala:15) [info]...

LiShuMing

These are currently missing

non

Not sure how applicable this is to Jawn, but should we look at using https://github.com/densh/scala-offheap to both improve the performance (but more importantly the latency/memory characteristics) of the parser?

mdedetrich

There are some smart strategies here: https://github.com/sirthias/benchmarks/blob/master/src/main/scala/benchmarks/json/ArrayBasedAstParser.scala We should steal some of them.

non

Make JObject and JArray immutable

8

Currently (and historically) `JObject` and `JArray` are mutable. They are case classes wrapping mutable data structures. @ijuma wisely points out that this is maybe not the best for folks who...

non

Faster whitespace detection

6

At nescala Matthias suggested there was a faster way to check for possible whitespace characters using bitwise logic. We should look into that.

non

## About this PR 📦 Updates * [org.scala-js:sbt-scalajs](https://github.com/scala-js/scala-js) * [org.scala-js:scalajs-compiler](https://github.com/scala-js/scala-js) * [org.scala-js:scalajs-library](https://github.com/scala-js/scala-js) * [org.scala-js:scalajs-library_2.13](https://github.com/scala-js/scala-js) * [org.scala-js:scalajs-test-bridge](https://github.com/scala-js/scala-js) * [org.scala-js:scalajs-test-bridge_2.13](https://github.com/scala-js/scala-js) from `1.15.0` to `1.16.0` 📜 [GitHub Release Notes](https://github.com/scala-js/scala-js/releases/tag/v1.16.0) - [Version Diff](https://github.com/scala-js/scala-js/compare/v1.15.0...v1.16.0) ##...

typelevel-steward[bot]

## About this PR 📦 Updates [org.scalameta:scalafmt-core](https://github.com/scalameta/scalafmt) from `3.8.0` to `3.8.1` 📜 [GitHub Release Notes](https://github.com/scalameta/scalafmt/releases/tag/v3.8.1) - [Version Diff](https://github.com/scalameta/scalafmt/compare/v3.8.0...v3.8.1) ## Usage ✅ **Please merge!** I'll automatically update this PR to resolve...

typelevel-steward[bot]

## About this PR 📦 Updates [org.scala-lang:scala-library](https://github.com/scala/scala) from `2.13.12` to `2.13.13` 📜 [GitHub Release Notes](https://github.com/scala/scala/releases/tag/v2.13.13) - [Version Diff](https://github.com/scala/scala/compare/v2.13.12...v2.13.13) ## Usage ✅ **Please merge!** I'll automatically update this PR to resolve...

typelevel-steward[bot]

Enable scala2 inliner

2

I noticed that this project uses the `@inline` annotation which actually doesn't do anything unless you have the scala-2 inliner enable, quoting from https://docs.scala-lang.org/overviews/compiler-options/optimizer.html > The `@inline` annotation only has...

mdedetrich