fktn-k / fkYAML

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

Fix parse error on block sequences with child flow style container nodes #332

Closed fktn-k closed 5 months ago

fktn-k commented 5 months ago

This PR has fixed parse errors which the current parser emits on a block sequence containing child flow style containers (either sequence and/or mapping), like the following valid YAML snippet (based on the official YAML test suite YD5X):

# child flow sequences
- [username, identifier, score]
- [booker12, 9012      , 61.25]
- [grey07  , 2070      , 84.50]

# child flow mappings
- {foo: false, null: 123}
- {true: 3.14, bar: [0x30,0o30]}

While modifying implementation to fix the above issue, the way of managing parse contexts has also been corrected by adding the variation of parse context states.


Pull Request Checklist

Read the CONTRIBUTING.md file for detailed information.

Please don't

github-actions[bot] commented 5 months ago

:octocat: Upload Coverage Event Notification

Coverage data has been uploaded for the commit 725bb6c3b560a9c4797223b094f5b4541a6c76d4. You can download the artifact which contains the same file uploaded to the Coveralls and its HTML version.

Name fkYAML_coverage.pr332.zip
ID 1465123755
URL https://github.com/fktn-k/fkYAML/actions/runs/8915291691/artifacts/1465123755
coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 8915291691

Details


Totals Coverage Status
Change from base Build 8899528587: 0.0%
Covered Lines: 3574
Relevant Lines: 3574

💛 - Coveralls