astutils.cpp: added more bailouts in isMutableExpression() [skip ci]

Author: firewave

lib/astutils.cpp

@@ -2458,9 +2458,11 @@ bool isMutableExpression(const Token* tok)
 {
     if (!tok)
         return false;
+    if (!tok->exprId())
+        return false;
     if (tok->isLiteral() || tok->isKeyword() || tok->isStandardType() || tok->isEnumerator())
         return false;
-    if (Token::Match(tok, ",|;|:|]|)|}"))
+    if (Token::Match(tok, ",|;|:|]|)|}|::"))
         return false;
     if (Token::simpleMatch(tok, "[ ]"))
         return false;
@@ -2847,6 +2849,8 @@ static std::function<R()> memoize(F f)
     };
 }
 
+#include <iostream>
+
 template<class F,
          REQUIRES("F must be a function that returns a Token class",
                   std::is_convertible<decltype(std::declval<F>()()), const Token*> )>
@@ -2880,7 +2884,10 @@ static bool isExpressionChangedAt(const F& getExprTok,
         if (!expr)
             aliased = true;
         if (!aliased)
+        {
+            std::cout << tok->str() << " " << tok->exprId() << std::endl;
             aliased = isAliasOf(tok, expr, &i);
+        }
         if (!aliased)
             return false;
         if (isVariableChanged(tok, indirect + i, settings, depth))