blues / note-c / 7701226127

93%

push

github

Make more JSON number handling improvements. (#136)

* Make more JSON number handling improvements.

- Incorporate Ray's improvements to n_ftoa.c and n_atof.c that allow us to
support larger exponents.
- Add JAtoN and JNtoA unit tests to test these improvements.
- If NOTE_C_LOW_MEM is defined, define JNUMBER to float. On truly low memory
platforms, this likely does nothing, because those platforms often map double
and float to the same underlying type (which is single-precision). However, when
running the unit tests on a CPU with double-precision floating point support,
this allows us to deliberately run the unit tests with JNUMBER set to
single-precision. Doing this broke a couple test cases in
JSON_number_handling_test.cpp, which I've fixed here.

* Add NOTE_C_TEST_SINGLE_PRECISION.

This `#define` forces `JNUMBER` to be `float`, rather than `double`. This allows
us to run the unit tests with single precision floats, even on a machine that
supports double precision. This is useful coverage because a lot of small MCUs
don't support double precision.

* Simplify JAtoN.

Remove the code blocks guarded by `#ifndef LARGE_EXPONENT_FIX` (i.e. make the
code behave like LARGE_EXPONENT_FIX is always defined). Remove the
LARGE_EXPONENT_FIX define.

988 of 1201 branches covered (0.0%)

Branch coverage included in aggregate %.

2109 of 2216 relevant lines covered (95.17%)

81.52 hits per line