search

francoispqt / gojay

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

EncodeBuilder: Remove support of controls that should not be supported in JSON strings #151

Closed marcsantiago closed 4 months 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 #151 into master will increase coverage by 0.03%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #151      +/-   ##
===========================================
+ Coverage   99.96%   100.00%   +0.03%     
===========================================
  Files          38        38              
  Lines        5323      5318       -5     
===========================================
- Hits         5321      5318       -3     
+ Misses          1         0       -1     
+ Partials        1         0       -1
Impacted Files Coverage Δ
encode_builder.go 100.00% <100.00%> (ø)
decode.go 100.00% <0.00%> (+0.68%) :arrow_up:

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...8d61161. Read the comment docs.