fktn-k / fkYAML / 10959179503

100%

push

github

Reduce string copies in parse (#394)

* implemented basic_str_view (as a minimal implementation of std::basic_string_view since C++17)

* modified the way of passing input buffer to lexer

* wip: add test for basic_str_view class

* use str_view to store YAML & TAG directive values

* deleted unused type aliases in lexer

* return str_view as part of lexical_token from get_next_token()

* added in-source documentations in str_view.hpp

* removed unnecessary template parameter for lexer

* removed unused header includes

* no copy when input buffer has contiguous bytes with no CRs

* rename fill_buffer() as get_buffer_view() in input_adapter variant classes

* added test cases for missing lines/branches

* updated benchmark scores

* fixed compile errors/warnings

* suppress deprecated declarations warning when test app is built with Apple Clang compilers

* fixed include path in test source file for str_view

* fixed outside of array bounds errors

* fixed issue on large std::string instance creation for test

* added explicit conversion to resolve warning when compiled with msvc

* skip str_view comparison check with large strings if int == ptrdiff_t

1254 of 1254 branches covered (100.0%)

Branch coverage included in aggregate %.

466 of 466 new or added lines in 9 files covered. (100.0%)

4573 of 4573 relevant lines covered (100.0%)

305.36 hits per line