Deserialization of numerics should follow general rounding rules

Issue #96 resolved
Francois Normandin created an issue

I’ve noticed that “from JSON text” ignores the decimal characters when deserializing to an integer.

Using the “Fract/Exp String to Number” instead of “Scan from String“ would fix the issue. (Use System Decimal Point = False)

Comments (6)

  1. James Powell repo owner

    That does make sense. Need to check for any performance cost of switching the function.

  2. James Powell repo owner

    Complication here is that arrays of integers are converted by the inbuilt JSON primitive (for speed reasons). Need to make sure that arrays and scalar integer conversion are consistent with each other (see also #93).

  3. James Powell repo owner

    Performance tests in LV2020 suggest this change is faster; fast enough to solve #93 and displace the problematic use of teh inbuilt JSON primitive for arrays.

  4. Log in to comment