z00m128 / sjasmplus / 1451

97%

master: 97%

Build Type

Pull #254

cirrus-ci

Committed by ped7g

Commit Message

c++17: replacing custom code with std::filesystem[::path] where feasible

Related #240, should fix "check include paths" feature for windows, and
simplify somewhat code base (and get rid of home-made filename patching
functions, they seem robust enough and well tested, but I assume in the
long term having C++ standard library implementation is better, even if
it involves slightly more machine code executed here and there).

This is not thorough refactoring of whole code base, working on it bit
by bit, luckily the std::filesystem API is flexible enough to
connect/disconnect it to rest of code through char[] fields, so I can
replace the code partially and check test results.

Commit v2 (replacing lot more variables and code than first proof of
concept).

Pull Request Pull Request #254: migrating some of the file stuff to std::filesystem, in the naive hope of getting better cross-platform compatibility

Run Details

77 of 85 new or added lines in 7 files covered. (90.59%)

38 existing lines in 6 files now uncovered.

9680 of 10054 relevant lines covered (96.28%)

168519.82 hits per line