-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
moved global CheckUnusedFunctions
instance to CppCheck
/ cleaned up CheckUnusedFunctions
interface
#6013
Conversation
@@ -96,9 +96,9 @@ class TestUnusedFunctions : public TestFixture { | |||
|
|||
// Check for unused functions.. | |||
CheckUnusedFunctions checkUnusedFunctions; | |||
checkUnusedFunctions.parseTokens(tokenizer, "someFile.c", settings1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another case of a language mismatch between tokenizing and usage.
Will file tickets for the |
@@ -560,6 +563,10 @@ unsigned int CppCheck::check(const std::string &path, const std::string &content | |||
|
|||
unsigned int CppCheck::check(const FileSettings &fs) | |||
{ | |||
// TODO: move to constructor when CppCheck no longer owns the settings |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yet another case highlighting how all the current messes I am working on are woven together...
This looks like a false positive:
If both sides are just a |
8e4fab9
to
09e5d2e
Compare
I filed https://trac.cppcheck.net/ticket/12455 about the |
@@ -0,0 +1,7 @@ | |||
static void f4_0() {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a gap in the test coverage which was uncovered by me not properly propagating all the data in an earlier version of these changes.
CheckUnusedFunctions
instance to CppCheck
/ cleaned up CheckUnusedFunctions
interfaceCheckUnusedFunctions
instance to CppCheck
/ cleaned up CheckUnusedFunctions
interface
09e5d2e
to
c68c0b1
Compare
…up `CheckUnusedFunctions` interface
c68c0b1
to
8e943c6
Compare
No description provided.