search

dlang-community / D-YAML

YAML parser and emitter for the D programming language
https://dlang-community.github.io/D-YAML/
Boost Software License 1.0
119 stars 37 forks source link

fix issue #257 #261

Closed Herringway closed 4 years ago

Herringway commented 4 years ago

The bounds checking on Reader.prefixBytes's contract was off by one. This would cause problems for loading strings with spaces at the end, but only in builds with contracts on. Fixes #257.

codecov[bot] commented 4 years ago

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #261   +/-   ##
=======================================
  Coverage   94.98%   94.98%           
=======================================
  Files          28       28           
  Lines        4126     4129    +3     
=======================================
+ Hits         3919     3922    +3     
  Misses        207      207
Impacted Files Coverage Δ
source/dyaml/reader.d 98.44% <100.00%> (+0.01%) :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 5171925...53d88a5. Read the comment docs.