search

francoispqt / gojay

high performance JSON encoder/decoder with stream API for Golang
MIT License
2.11k stars 113 forks source link

EncodeBuilder: Correcting which type of controls should be written in a JSON string #149

Closed marcsantiago closed 4 years ago

marcsantiago commented 4 years ago

According to the JSON spec these are the only valid and considered controls for JSON

\b Backspace (ascii code 08) \f Form feed (ascii code 0C) \n New line \r Carriage return \t Tab \" Double quote \ Backslash character https://www.json.org/json-en.html

fixes https://github.com/francoispqt/gojay/issues/145

codecov-commenter commented 4 years ago

Codecov Report

Merging #149 into master will increase coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   99.96%   99.96%           
=======================================
  Files          38       39    +1     
  Lines        5323     5334   +11     
=======================================
+ Hits         5321     5332   +11     
  Misses          1        1           
  Partials        1        1
Impacted Files Coverage Δ
decode_bool.go 100.00% <100.00%> (ø)
decode_number_int.go 100.00% <100.00%> (ø)
decode_token.go 100.00% <100.00%> (ø)
encode_builder.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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