fktn-k / fkYAML

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

Normalize newline codes in the input buffer #343

Closed fktn-k closed 5 months ago

fktn-k commented 5 months ago

This PR has added the feature of normalizing newline codes in input buffers (LF and CR+LF codes to LF) in the input_adapter module and thus removed the handling of CR+LF codes implemented in the lexical_analyzer class.
The change will improve performance in scanning input buffers in the lexical_analyzer class since it no longer needs to take into account the existence of CR+LF codes contained in the target buffer.
The test suite has also been updated to validate the normalization feature.


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 96f3cd21b4d06d7969f6a87871388c4dbffc0b02. You can download the artifact which contains the same file uploaded to the Coveralls and its HTML version.

Name fkYAML_coverage.pr343.zip
ID 1516478399
URL https://github.com/fktn-k/fkYAML/actions/runs/9144636433/artifacts/1516478399
coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9144636433

Details


Totals Coverage Status
Change from base Build 9115035026: 0.0%
Covered Lines: 3700
Relevant Lines: 3700

💛 - Coveralls