Closed rpoyner-tri closed 2 years ago
@SimonKagstrom : feel free to use and/or improve on this patch. The test covers the situation described in #368, but could probably be improved. I was able to work out how to run the tests locally, and verified passing-with-fix, and failing-without fix.
Merging #369 (d00ed59) into master (8419da5) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #369 +/- ##
=======================================
Coverage 63.45% 63.45%
=======================================
Files 57 57
Lines 4553 4553
=======================================
Hits 2889 2889
Misses 1664 1664
Impacted Files | Coverage Δ | |
---|---|---|
src/configuration.cc | 54.77% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 8419da5...d00ed59. Read the comment docs.
Merged! Thanks for taking the time to write the regression test as well!
Glad to be of service. Been using kcov for many years; thanks for your work on it!
Happy to hear that, thanks a lot!
Closes: #368