Token: removed test-only stringifyList() overload (#8411)

several other tests already use the multi-parameter version

Author: firewave

lib/token.cpp

@@ -1421,11 +1421,6 @@ std::string Token::stringifyList(const Token* end, bool attributes) const
     return stringifyList(false, attributes, false, false, false, nullptr, end);
 }
 
-std::string Token::stringifyList(bool varid) const
-{
-    return stringifyList(varid, false, true, true, true, nullptr, nullptr);
-}
-
 void Token::astParent(Token* tok)
 {
     const Token* tok2 = tok;

lib/token.h

@@ -1110,7 +1110,7 @@ class CPPCHECKLIB Token {
             options.files = true;
             return options;
         }
-        // cppcheck-suppress unusedFunction
+        // cppcheck-suppress unusedFunction - used in tests only
         static stringifyOptions forDebugVarId() {
             stringifyOptions options = forDebug();
             options.varid = true;
@@ -1142,7 +1142,6 @@ class CPPCHECKLIB Token {
 
     std::string stringifyList(const stringifyOptions& options, const std::vector<std::string>* fileNames = nullptr, const Token* end = nullptr) const;
     std::string stringifyList(const Token* end, bool attributes = true) const;
-    std::string stringifyList(bool varid = false) const;
 
     /**
      * Stringify a list of token, from current instance on.

test/testsimplifytokens.cpp

@@ -208,7 +208,7 @@ class TestSimplifyTokens : public TestFixture {
         ASSERT_LOC(tokenizer.tokenize(code), file, line);
 
         // result..
-        return tokenizer.tokens()->stringifyList(true);
+        return tokenizer.tokens()->stringifyList(true, false, true, true, true);
     }
 
 

test/testtokenize.cpp

@@ -8378,7 +8378,7 @@ class TestTokenizer : public TestFixture {
         // Tokenizer..
         ASSERT_LOC(tokenizer.simplifyTokens1(""), file, line);
 
-        return tokenizer.tokens()->stringifyList();
+        return tokenizer.tokens()->stringifyList(false, false, true, true, true);
     }
 
     void checkHeader1() {