biojppm / rapidyaml / 583

93%

master: 93%

push

travis-ci

work around weird g++ optimizer error when emitting null scalars

... should emit "~" but was emitting "''". This happened because gcc was
returning "s.str != nullptr" as false even when s.str was nullptr (!!!).
Decided to move the handling of the empty strings to the beginning of the
function (earlier than the needs_quotes assignment) to avoid throwing off the
branch predictor off. This happened only on Release builds. g++10 was fine,
but g++-9 and earlier were wrong. clang or visual studio succeeded here.
See the logs: https://travis-ci.org/github/biojppm/rapidyaml/builds/730368478

6 of 6 new or added lines in 1 file covered. (100.0%)

3845 of 4121 relevant lines covered (93.3%)

388711.35 hits per line