Format

Author: pfultz2

lib/astutils.cpp

@@ -429,9 +429,10 @@ bool isStlStringType(const Token* tok)
            (Token::simpleMatch(tok, "std :: basic_string <") && !Token::simpleMatch(tok->linkAt(3), "> ::"));
 }
 
-bool isVoidCast(const Token *tok)
+bool isVoidCast(const Token* tok)
 {
-    return Token::simpleMatch(tok, "(") && tok->isCast() && tok->valueType() && tok->valueType()->type == ValueType::Type::VOID && tok->valueType()->pointer == 0;
+    return Token::simpleMatch(tok, "(") && tok->isCast() && tok->valueType() &&
+           tok->valueType()->type == ValueType::Type::VOID && tok->valueType()->pointer == 0;
 }
 
 bool isTemporary(const Token* tok, const Library* library, bool unknown)

lib/astutils.h

@@ -188,7 +188,7 @@ const Token * astIsVariableComparison(const Token *tok, const std::string &comp,
 bool isVariableDecl(const Token* tok);
 bool isStlStringType(const Token* tok);
 
-bool isVoidCast(const Token *tok);
+bool isVoidCast(const Token* tok);
 
 bool isTemporary(const Token* tok, const Library* library, bool unknown = false);
 

lib/checkother.cpp

@@ -1643,14 +1643,14 @@ static bool isVariableMutableInInitializer(const Token* start, const Token * end
 
 static bool isCastToVoid(const Variable* var)
 {
-    if(!var)
+    if (!var)
         return false;
-    if(!var->scope())
+    if (!var->scope())
         return false;
-    for (const Token *tok = var->scope()->bodyStart; tok != var->scope()->bodyEnd; tok = tok->next()) {
-        if(tok->varId() != var->declarationId())
+    for (const Token* tok = var->scope()->bodyStart; tok != var->scope()->bodyEnd; tok = tok->next()) {
+        if (tok->varId() != var->declarationId())
             continue;
-        if(isVoidCast(tok->astParent()))
+        if (isVoidCast(tok->astParent()))
             return true;
     }
     return false;

test/testother.cpp

@@ -12164,8 +12164,7 @@ class TestOther : public TestFixture {
               "        for (auto &j : g(std::move(l))) { (void)j; }\n"
               "    }\n"
               "}\n");
-        ASSERT_EQUALS("[test.cpp:4:36]: (warning) Access of moved variable 'l'. [accessMoved]\n",
-                      errout_str());
+        ASSERT_EQUALS("[test.cpp:4:36]: (warning) Access of moved variable 'l'. [accessMoved]\n", errout_str());
     }
 
     void moveCallback()