mitigated unusedFunction nogui selfcheck warnings

firewave · firewave · commit b7383c568bd8 · 2024-06-14T14:03:50.000+02:00
diff --git a/lib/checkleakautovar.cpp b/lib/checkleakautovar.cpp
@@ -104,47 +104,6 @@ static bool isVarTokComparison(const Token * tok, const Token ** vartok,
 
 //---------------------------------------------------------------------------
 
-void VarInfo::print()
-{
-    std::cout << "size=" << alloctype.size() << std::endl;
-    for (std::map<int, AllocInfo>::const_iterator it = alloctype.cbegin(); it != alloctype.cend(); ++it) {
-        std::string strusage;
-        const auto use = possibleUsage.find(it->first);
-        if (use != possibleUsage.end())
-            strusage = use->second.first->str();
-
-        std::string status;
-        switch (it->second.status) {
-        case OWNED:
-            status = "owned";
-            break;
-        case DEALLOC:
-            status = "dealloc";
-            break;
-        case ALLOC:
-            status = "alloc";
-            break;
-        case NOALLOC:
-            status = "noalloc";
-            break;
-        case REALLOC:
-            status = "realloc";
-            break;
-        default:
-            status = "?";
-            break;
-        }
-
-        std::cout << "status=" << status << " "
-                  << "alloctype='" << it->second.type << "' "
-                  << "possibleUsage='" << strusage << "' "
-                  << "conditionalAlloc=" << (conditionalAlloc.find(it->first) != conditionalAlloc.end() ? "yes" : "no") << " "
-                  << "referenced=" << (referenced.find(it->first) != referenced.end() ? "yes" : "no") << " "
-                  << "reallocedFrom=" << it->second.reallocedFromType
-                  << std::endl;
-    }
-}
-
 void VarInfo::possibleUsageAll(const std::pair<const Token*, Usage>& functionUsage)
 {
     possibleUsage.clear();
diff --git a/lib/checkleakautovar.h b/lib/checkleakautovar.h
@@ -93,8 +93,6 @@ class CPPCHECKLIB VarInfo {
 
     /** set possible usage for all variables */
     void possibleUsageAll(const std::pair<const Token*, Usage>& functionUsage);
-
-    void print();
 };
 
 
diff --git a/lib/errorlogger.h b/lib/errorlogger.h
@@ -186,6 +186,7 @@ class CPPCHECKLIB ErrorMessage {
     }
 
     /** Verbose message (may be the same as the short message) */
+    // cppcheck-suppress unusedFunction - used by GUI only
     const std::string &verboseMessage() const {
         return mVerboseMessage;
     }
diff --git a/lib/importproject.cpp b/lib/importproject.cpp
@@ -1324,6 +1324,7 @@ void ImportProject::selectOneVsConfig(Platform::Type platform)
     }
 }
 
+// cppcheck-suppress unusedFunction - used by GUI only
 void ImportProject::selectVsConfigurations(Platform::Type platform, const std::vector<std::string> &configurations)
 {
     for (std::list<FileSettings>::iterator it = fileSettings.begin(); it != fileSettings.end();) {
@@ -1348,6 +1349,7 @@ void ImportProject::selectVsConfigurations(Platform::Type platform, const std::v
     }
 }
 
+// cppcheck-suppress unusedFunction - used by GUI only
 std::list<std::string> ImportProject::getVSConfigs()
 {
     return std::list<std::string>(mAllVSConfigs.cbegin(), mAllVSConfigs.cend());
diff --git a/lib/suppressions.cpp b/lib/suppressions.cpp
@@ -390,6 +390,7 @@ bool SuppressionList::Suppression::isMatch(const SuppressionList::ErrorMessage &
     return true;
 }
 
+// cppcheck-suppress unusedFunction - used by GUI only
 std::string SuppressionList::Suppression::getText() const
 {
     std::string ret;

Original file line number	Diff line number	Diff line change
`@@ -186,6 +186,7 @@ class CPPCHECKLIB ErrorMessage {`
`186`	`186`	`}`
`187`	`187`
`188`	`188`	`/** Verbose message (may be the same as the short message) */`
	`189`	`+ // cppcheck-suppress unusedFunction - used by GUI only`
`189`	`190`	`const std::string &verboseMessage() const {`
`190`	`191`	`return mVerboseMessage;`
`191`	`192`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1324,6 +1324,7 @@ void ImportProject::selectOneVsConfig(Platform::Type platform)`
`1324`	`1324`	`}`
`1325`	`1325`	`}`
`1326`	`1326`
	`1327`	`+// cppcheck-suppress unusedFunction - used by GUI only`
`1327`	`1328`	`void ImportProject::selectVsConfigurations(Platform::Type platform, const std::vector<std::string> &configurations)`
`1328`	`1329`	`{`
`1329`	`1330`	`for (std::list<FileSettings>::iterator it = fileSettings.begin(); it != fileSettings.end();) {`
`@@ -1348,6 +1349,7 @@ void ImportProject::selectVsConfigurations(Platform::Type platform, const std::v`
`1348`	`1349`	`}`
`1349`	`1350`	`}`
`1350`	`1351`
	`1352`	`+// cppcheck-suppress unusedFunction - used by GUI only`
`1351`	`1353`	`std::list<std::string> ImportProject::getVSConfigs()`
`1352`	`1354`	`{`
`1353`	`1355`	`return std::list<std::string>(mAllVSConfigs.cbegin(), mAllVSConfigs.cend());`
Original file line number	Diff line number	Diff line change
`@@ -390,6 +390,7 @@ bool SuppressionList::Suppression::isMatch(const SuppressionList::ErrorMessage &`
`390`	`390`	`return true;`
`391`	`391`	`}`
`392`	`392`
	`393`	`+// cppcheck-suppress unusedFunction - used by GUI only`
`393`	`394`	`std::string SuppressionList::Suppression::getText() const`
`394`	`395`	`{`
`395`	`396`	`std::string ret;`