From 6f931825804becc6e047ba2e685b6698321c9508 Mon Sep 17 00:00:00 2001 From: Slava Semushin Date: Sun, 30 Aug 2009 02:03:37 +0700 Subject: [PATCH] Tokenizer::simplifyLogicalOperators(): change return type to void. No functional change. --- src/tokenize.cpp | 4 +--- src/tokenize.h | 4 +--- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/src/tokenize.cpp b/src/tokenize.cpp index 0d29fa258..33505731b 100644 --- a/src/tokenize.cpp +++ b/src/tokenize.cpp @@ -2819,11 +2819,10 @@ bool Tokenizer::simplifyIfNotNull() } -bool Tokenizer::simplifyLogicalOperators() +void Tokenizer::simplifyLogicalOperators() { // "if (not p)" => "if (!p)" // "if (p and q)" => "if (p and q)" - bool ret = false; for (Token *tok = _tokens; tok; tok = tok->next()) { if (Token::Match(tok, "if|while ( not %var%")) @@ -2846,7 +2845,6 @@ bool Tokenizer::simplifyLogicalOperators() tok->str("&&"); } } - return ret; } diff --git a/src/tokenize.h b/src/tokenize.h index f5a619c76..a5ba86e82 100644 --- a/src/tokenize.h +++ b/src/tokenize.h @@ -167,10 +167,8 @@ private: * Examples: * "if (not p)" => "if (!p)" * "if (p and q)" => "if (p && q)" - * @return true if something is modified - * false if nothing is done. */ - bool simplifyLogicalOperators(); + void simplifyLogicalOperators(); /** * Simplify comma into a semicolon when possible