testrunner: start adding IDs and columns to error messages by firewave · Pull Request #7343 · cppcheck-opensource/cppcheck
Requires #4377 and #7342 to be merged first.
This will help with identifying missing test coverage. Those should be collected below https://trac.cppcheck.net/ticket/12232.
It will also make clearer what is being tested. And would allow to re-group/limit of of the tests (in case someone really got absolutely nothing else to do).
This comment was marked as outdated.
This comment was marked as outdated.
testrunner: start adding IDs to error messages
testrunner: start adding IDs and columns to error messages
| " { }\n" | ||
| "}"); | ||
| ASSERT_EQUALS("[test.cpp:1]: (style) The function 'f1' is never used.\n", errout_str()); | ||
| ASSERT_EQUALS("[test.cpp:1:0]: (style) The function 'f1' is never used. [unusedFunction]\n", errout_str()); |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These should probably have a proper column.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
