fktn-k / fkYAML

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

Scan inputs using iterators #306

Closed fktn-k closed 6 months ago

fktn-k commented 6 months ago

This PR refactored the processing of input characters from reading them one byte at a time from the input_handler object to accessing the input buffer using an iterator in the lexer.
This change reduced the number of method calls to retrieve lexical tokens (especially string scalars) from the input buffer.
In the process of accomplishing this, the input_handler class was replaced by the newly implemented position_tracker class.
During the above refactoring, the input buffer scanning method was also improved.


Pull Request Checklist

Read the CONTRIBUTING.md file for detailed information.

Please don't

github-actions[bot] commented 6 months ago

:octocat: Upload Coverage Event Notification

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

Name fkYAML_coverage.pr306.zip
ID 1394689183
URL https://github.com/fktn-k/fkYAML/actions/runs/8602562456/artifacts/1394689183
coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 8602562456

Details


Totals Coverage Status
Change from base Build 8582568658: 0.0%
Covered Lines: 3464
Relevant Lines: 3464

💛 - Coveralls