TOKEN: Patterns like "if ; !!else" are now possible

testtokenize.cpp

@@ -52,6 +52,8 @@ private:
 
         TEST_CASE( match1 );
 
+        TEST_CASE( match2 );
+
         TEST_CASE( varid1 );
     }
 
@@ -263,6 +265,68 @@ private:
         }
     }
 
+    void match2()
+    {
+        {
+            const std::string code("");
+
+            // tokenize..
+            Tokenizer tokenizer;
+            std::istringstream istr(code);
+            tokenizer.tokenize(istr, "test.cpp");
+
+            // Match..
+            ASSERT_EQUALS( true, TOKEN::Match(tokenizer.tokens(), "!!else") );
+        }
+
+        {
+            const std::string code("if ;");
+
+            // tokenize..
+            Tokenizer tokenizer;
+            std::istringstream istr(code);
+            tokenizer.tokenize(istr, "test.cpp");
+
+            // Match..
+            ASSERT_EQUALS( true, TOKEN::Match(tokenizer.tokens(), "if ; !!else") );
+        }
+
+        {
+            const std::string code("if ; something");
+
+            // tokenize..
+            Tokenizer tokenizer;
+            std::istringstream istr(code);
+            tokenizer.tokenize(istr, "test.cpp");
+
+            // Match..
+            ASSERT_EQUALS( true, TOKEN::Match(tokenizer.tokens(), "if ; !!else") );
+        }
+
+        {
+            const std::string code("else");
+
+            // tokenize..
+            Tokenizer tokenizer;
+            std::istringstream istr(code);
+            tokenizer.tokenize(istr, "test.cpp");
+
+            // Match..
+            ASSERT_EQUALS( false, TOKEN::Match(tokenizer.tokens(), "!!else") );
+        }
+
+        {
+            const std::string code("if ; else");
+
+            // tokenize..
+            Tokenizer tokenizer;
+            std::istringstream istr(code);
+            tokenizer.tokenize(istr, "test.cpp");
+
+            // Match..
+            ASSERT_EQUALS( false, TOKEN::Match(tokenizer.tokens(), "if ; !!else") );
+        }
+    }
 
     void varid1()
     {

token.cpp

@@ -142,12 +142,8 @@ int TOKEN::multiCompare( const char *needle, const char *haystack )
     return -1;
 }
 
-
 bool TOKEN::Match(const TOKEN *tok, const char pattern[], const char *varname1[], const char *varname2[], unsigned int varid)
 {
-    if (!tok)
-        return false;
-
     const char *p = pattern;
     while ( *p )
     {
@@ -155,6 +151,15 @@ bool TOKEN::Match(const TOKEN *tok, const char pattern[], const char *varname1[]
         while ( *p == ' ' )
             p++;
 
+        if (!tok)
+        {
+            // If we have no tokens, pattern "!!else" should return true
+            if( isNotPattern( p ) )
+                return true;
+            else
+                return false;
+        }
+
         // Extract token from pattern..
         // TODO: Refactor this so there can't be buffer overflows
         char str[500];
@@ -254,18 +259,31 @@ bool TOKEN::Match(const TOKEN *tok, const char pattern[], const char *varname1[]
             }
         }
 
+        // Parse "not" options. Token can be anything except the given one
+        else if( isNotPattern( str ) )
+        {
+            if( strcmp( tok->aaaa(), &(str[2]) ) == 0 )
+                return false;
+        }
+
         else if (str != tok->_str)
             return false;
 
         tok = tok->next();
-        if (!tok && *p)
-            return false;
     }
 
     // The end of the pattern has been reached and nothing wrong has been found
     return true;
 }
 
+bool TOKEN::isNotPattern( const char *pattern )
+{
+    if( pattern && strlen(pattern) > 2 && pattern[0] == '!' && pattern[1] == '!' )
+        return true;
+    else
+        return false;
+}
+
 bool TOKEN::isName() const
 {
     return _isName;

token.h

@@ -75,6 +75,7 @@ public:
      * "[abc]" Any of the characters 'a' or 'b' or 'c'
      * "int|void|char" Any of the strings, int, void or char
      * "int|void|char|" Any of the strings, int, void or char or empty string
+     * "!!else" No tokens or any token that is not "else".
      * "someRandomText" If token contains "someRandomText".
      *
      * The patterns can be also combined to compare to multiple tokens at once
@@ -159,6 +160,17 @@ public:
 private:
     void next( TOKEN *next );
     void previous( TOKEN *previous );
+
+    /**
+     * Return true if pattern is e.g. "!!else".
+     * See Match() for more info.
+     *
+     * @param pattern Pattern to match, e.g. "if ; !!else"
+     * @return true if pattern starts with "!!" and contains 3
+     * or more characters.
+     */
+    static bool isNotPattern( const char *pattern );
+
     std::string _str;
     char * _cstr;
     bool _isName;