Niels Lohmann
Niels Lohmann
Once the pipeline is through, please check why the coverage went down. Note you can download an artifact from the coverage job which contains HTML pages showing which lines are...
These lines can be skipped by adding `// LCOV_EXCL_LINE`.
Please update to the latest `develop` branch which contains a fix for Clang-Tidy, see #4558.
Thanks a lot!
@sushshring I overworked the documentation of this feature a bit. Could you please have a look at - https://github.com/nlohmann/json/blob/cleanup/docs/mkdocs/docs/api/macros/json_diagnostic_positions.md - https://github.com/nlohmann/json/blob/cleanup/docs/mkdocs/docs/api/basic_json/start_pos.md - https://github.com/nlohmann/json/blob/cleanup/docs/mkdocs/docs/api/basic_json/end_pos.md If you have any comments, please add...
@alainsanguinetti Can you provide an updated list of the results?
Thanks! I will see what's still relevant from this.
Thanks @gregmarr for the analysis! - The issue with not moving a parser_callback_t should be fixed with #4489 - I agree with your analysis on resource leaks and the misunderstanding...
Any update on this?
Please run `make amalgamate` with AStyle 3.1.