Improve ast generation for templated function parameters (#2803)

lib/checkunusedvar.cpp

@@ -1229,6 +1229,9 @@ void CheckUnusedVar::checkFunctionVariableUsage()
 
             const Token *expr = varDecl ? varDecl : tok->astOperand1();
 
+            if (isInitialization)
+                expr = tok->previous();
+
             // Is variable in lhs a union member?
             if (tok->previous() && tok->previous()->variable() && tok->previous()->variable()->nameToken()->scope()->type == Scope::eUnion)
                 continue;

lib/tokenlist.cpp

@@ -796,8 +796,19 @@ static void compileTerm(Token *&tok, AST_state& state)
             }
         } else if (!state.cpp || !Token::Match(tok, "new|delete %name%|*|&|::|(|[")) {
             tok = skipDecl(tok);
-            while (tok->next() && tok->next()->isName())
-                tok = tok->next();
+            bool repeat = true;
+            while (repeat)
+            {
+                repeat = false;
+                if (Token::Match(tok->next(), "%name%")) {
+                    tok = tok->next();
+                    repeat = true;
+                }
+                if (Token::simpleMatch(tok->next(), "<") && Token::Match(tok->linkAt(1), "> %name%")) {
+                    tok = tok->next()->link()->next();
+                    repeat = true;
+                }
+            }
             state.op.push(tok);
             if (Token::Match(tok, "%name% <") && tok->linkAt(1))
                 tok = tok->linkAt(1);

test/testtokenize.cpp

@@ -494,6 +494,8 @@ private:
         TEST_CASE(noCrash1);
         TEST_CASE(noCrash2);
 
+        TEST_CASE(noCrash3);
+
         // --check-config
         TEST_CASE(checkConfiguration);
 
@@ -8519,6 +8521,10 @@ private:
                             "template <> d<c>::d(const d &) {}\n"));
     }
 
+    void noCrash3() {
+        ASSERT_NO_THROW(tokenizeAndStringify("void a(X<int> x, typename Y1::Y2<int, A::B::C, 2> y, Z z = []{});"));
+    }
+
     void checkConfig(const char code[]) {
         errout.str("");