From 6a83e24c9080081e1c1ca640ad2ec9b7a605f161 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Marjam=C3=A4ki?= Date: Mon, 29 Dec 2008 08:12:33 +0000 Subject: [PATCH] Memory leak : Removed checking of 'if(true)' and 'if(false)' as these are handled in the Tokenizer --- checkmemoryleak.cpp | 17 +---------------- 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/checkmemoryleak.cpp b/checkmemoryleak.cpp index eb5c1f320..56e8199aa 100644 --- a/checkmemoryleak.cpp +++ b/checkmemoryleak.cpp @@ -467,14 +467,6 @@ TOKEN *CheckMemoryLeakClass::getcode(const TOKEN *tok, std::list { addtoken("if(!var)"); } - else if ( TOKEN::Match(tok, "if ( true )") ) - { - addtoken("if(true)"); - } - else if ( TOKEN::Match(tok, "if ( false )") ) - { - addtoken("if(false)"); - } else if ( TOKEN::Match(tok, "if") ) { // Check if the condition depends on var somehow.. @@ -821,7 +813,7 @@ void CheckMemoryLeakClass::simplifycode(TOKEN *tok) } // Reduce "if* ;" that is not followed by an else.. - if (TOKEN::Match(tok2->next(), "if(var)|if(!var)|if(true)|if(false)|ifv ; !!else") ) + if (TOKEN::Match(tok2->next(), "if(var)|if(!var)|ifv ; !!else") ) { erase(tok2, tok2->tokAt(2)); done = false; @@ -866,13 +858,6 @@ void CheckMemoryLeakClass::simplifycode(TOKEN *tok) done = false; } - // Reduce "if(true) X ;" => "X ;" - if (TOKEN::Match(tok2->next(), "if(true) %var% ; !!else") ) - { - erase( tok2, tok2->tokAt(2) ); - done = false; - } - if ( TOKEN::simpleMatch(tok2->next(), "loop") ) { // Replace "loop ;" with ";"