fktn-k / fkYAML

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

Fixed missing callings of `fclose()` in the unit tests #246

Closed fktn-k closed 10 months ago

fktn-k commented 10 months ago

I've noticed that the test with Valgrind emits warnings about possible memory leaks.
Although it's turned out missing callings of the std::fclose() function in the unit tests for encode detector is the cause and that doesn't affect the fkYAML library itself, it should be better to fix them.
No other fixes are included in this PR.


Pull Request Checklist

Read the CONTRIBUTING.md file for detailed information.

Please don't

coveralls commented 10 months ago

Coverage Status

coverage: 100.0%. remained the same when pulling 719a553d4d4f2a773336a3c24a8599c1a76ffbcd on bugfix/fix_valgrind_memory_leaks_warnings into e98d1d0fee114c76a998f5f1191bd9f6997bfc95 on develop.