Feature: serialize special numbers #884

grisumbras · 2023-05-07T18:26:50Z

Change serializer to output special floating point numbers differently, so that the output is always valid JSON.
- Infinity is output as 1e99999 (this number is so big that it is parsed by every implementation I am aware of as FP infinity).
- NaN is output as null. It changes the type, and roundtrip won't happen, but at least it's not something that is not JSON.
Add serializer option to enable an extension (the extension is incidentally the current behaviour)
- Infinity is output as Infinity.
- NaN is output as NaN.
Add the ability to control serialisation options when outputting into ostreams.

The benefit for the second behaviour is that infinity and NaN are using unambiguous special representation.

codecov · 2023-05-07T18:43:28Z

Codecov Report

Merging #884 (a4ecd74) into develop (ac9080c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head a4ecd74 differs from pull request most recent head 6a12c32. Consider uploading reports for the commit 6a12c32 to get more accurate results

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #884   +/-   ##
========================================
  Coverage    99.24%   99.24%           
========================================
  Files           71       71           
  Lines         6861     6889   +28     
========================================
+ Hits          6809     6837   +28     
  Misses          52       52

Impacted Files	Coverage Δ
include/boost/json/serializer.hpp	`100.00% <ø> (ø)`
include/boost/json/detail/impl/format.ipp	`100.00% <100.00%> (ø)`
include/boost/json/detail/ryu/detail/common.hpp	`100.00% <100.00%> (ø)`
include/boost/json/detail/ryu/impl/d2s.ipp	`99.55% <100.00%> (+<0.01%)`	⬆️
include/boost/json/impl/serialize.ipp	`95.60% <100.00%> (+0.66%)`	⬆️
include/boost/json/impl/serializer.ipp	`98.63% <100.00%> (+<0.01%)`	⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ac9080c...6a12c32. Read the comment docs.