Merged PR #10

Thanks @Farigh

Author: kunaltyagi

rules/RULE_3_3_A_start_function_name_with_is_or_has_when_return_bool.py

@@ -1,6 +1,6 @@
 """
 Start the function name with 'is' or 'has' when returning bool.
-This rule might be violated a lot because there are a lot of cases 
+This rule might be violated a lot because there are a lot of cases
 in which functions return bool to indicate exceptions.
 Please turn off this rule. If you think it's too overwhelming.
 
@@ -9,13 +9,13 @@
     bool checkSth() { <== Violation. The function name should be isSth or hasSth.
         return false;
     }
-    
+
 == Good ==
 
     bool isSth() { <== OK.
         return true;
     }
-    
+
     is isSth() { <== Don't care. it's not returnning bool.
     }
 """
@@ -39,7 +39,7 @@ def RunRule(lexer, fullName, decl, contextStack, context) :
                        "The function name(%s) should start with has or is when returinning bool" % fullName)
             break;
         k += 1
-         
+
 ruleManager.AddFunctionNameRule(RunRule)
 
 
@@ -59,44 +59,44 @@ def RunRule(lexer, fullName, decl, contextStack, context) :
 class testRule(nct):
     def setUpRule(self):
         ruleManager.AddFunctionNameRule(RunRule)
-    
+
     def test1(self):
-        self.Analyze("test/thisFile.c", 
+        self.Analyze("test/thisFile.c",
 """
 bool canHave() {
 }""")
-        assert CheckErrorContent(__name__)    
+        assert CheckErrorContent(__name__)
     def test2(self):
-        self.Analyze("test/thisFile.c", 
+        self.Analyze("test/thisFile.c",
 """
 bool CTEST:canHave() {
 }""")
-        assert CheckErrorContent(__name__)    
+        assert CheckErrorContent(__name__)
     def test3(self):
-        self.Analyze("test/thisFile.c", 
+        self.Analyze("test/thisFile.c",
 """
 extern bool CTEST:canHave() {
 }""")
-        assert CheckErrorContent(__name__)    
+        assert CheckErrorContent(__name__)
     def test4(self):
-        self.Analyze("test/thisFile.c", 
+        self.Analyze("test/thisFile.c",
 """
 extern int CTEST:canHave() {
 }""")
-        assert not CheckErrorContent(__name__)    
+        assert not CheckErrorContent(__name__)
     def test5(self):
-        self.Analyze("test/thisFile.c", 
+        self.Analyze("test/thisFile.c",
 """
 extern int CTEST:isIt() {
 }""")
-        assert not CheckErrorContent(__name__)    
+        assert not CheckErrorContent(__name__)
     def test6(self):
-        self.Analyze("test/thisFile.c", 
+        self.Analyze("test/thisFile.c",
 """
 class K {
 extern bool CTEST:canHave();
 }""")
-        assert CheckErrorContent(__name__)    
+        assert CheckErrorContent(__name__)
 
     def test7(self):
         self.Analyze("test/thisFile.c", """
@@ -112,18 +112,22 @@ def test7(self):
               */
              void AddGate(GATE gate){m_GateCont.push_back(gate); }
 """)
-        assert not CheckErrorContent(__name__)    
-        
+        assert not CheckErrorContent(__name__)
+
     def test8(self):
-        self.Analyze("test/thisFile.c", 
+        self.Analyze("test/thisFile.c",
 """
 boolean operator=();
 boolean KK::operator=();
 """)
         assert not CheckErrorContent(__name__)
+
     def test9(self):
         self.Analyze("test/thisFile.c",
 """
+/**
+  * This tests for correct parsing of fn name and nested templates
+  **/
 template<class ObjectTypePtr,
          typename = typename std::enable_if<std::is_pointer<ObjectTypePtr>::value>::type>
 bool canHave(ObjectTypePtr obj) {
@@ -134,9 +138,13 @@ def test9(self):
 bool canHave(ObjectTypeNotPtr obj) {
 }""")
         assert CheckErrorContent(__name__)
+
     def test10(self):
         self.Analyze("test/thisFile.c",
 """
+/**
+  * This tests for correct parsing of fn name and nested templates
+  **/
 template<class ObjectTypePtr,
          typename = typename std::enable_if<std::is_pointer<ObjectTypePtr>::value>::type>
 bool isIt(ObjectTypePtr obj) {