fktn-k / fkYAML

A C++ header-only YAML library
MIT License
69 stars 7 forks source link

#288 Fixed incorrect parse results from mapping entries split across newlines #289

Closed fktn-k closed 7 months ago

fktn-k commented 7 months ago

As reported in the issue #288, the the parser & lexer implementations didn't cover mapping entries split across newlines.
To fix that, the handling of key separators (:) have been moved from the lexer to the deserializer, and defer checking the existence of a key separator followed by a node associated to a block mapping key.
To validate the above changes, the test suite has been updated with some new test cases.


Pull Request Checklist

Read the CONTRIBUTING.md file for detailed information.

Please don't

coveralls commented 7 months ago

Coverage Status

coverage: 100.0%. remained the same when pulling 21e7f900ddea9d3a3fabd871f79d4b3f33a4c18a on bugfix/288_parse_mapping_entries_split_across_newlines into d77e7f58b68970b6a12c67dde6e2569863bc7328 on develop.