mitigated pylint warnings

Author: firewave

addons/cppcheckdata.py

@@ -677,7 +677,7 @@ def __init__(self, element, nestedIn):
         self.argumentId = {}
 
     def __repr__(self):
-        attrs = ["Id", "tokenId", "tokenDefId", "name", "type", "hasVirtualSpecifier", 
+        attrs = ["Id", "tokenId", "tokenDefId", "name", "type", "hasVirtualSpecifier",
                  "isImplicitlyVirtual", "access", "isInlineKeyword", "isStatic", 
                  "isAttributeNoreturn", "overriddenFunction", "nestedIn", "argumentId"]
         return "{}({})".format(
@@ -901,7 +901,7 @@ def setId(self, IdMap):
         self.symbolic = IdMap.get(self._symbolicId)
 
     def __repr__(self):
-        attrs = ["intvalue", "tokvalue", "floatvalue", "movedvalue", "uninit", 
+        attrs = ["intvalue", "tokvalue", "floatvalue", "movedvalue", "uninit",
                  "bufferSize", "containerSize", "condition", "valueKind"]
         return "{}({})".format(
             "Value",
@@ -999,7 +999,7 @@ def isMatch(self, file, line, message, errorId):
             return True
         # Other Suppression (Globaly set via suppression file or cli command)
         if ((self.fileName is None or fnmatch(file, self.fileName))
-                and (self.suppressionType is None) 
+                and (self.suppressionType is None)
                 and (self.symbolName is None or fnmatch(message, '*'+self.symbolName+'*'))
                 and fnmatch(errorId, self.errorId)):
             return True

addons/misra.py

@@ -2105,7 +2105,7 @@ def misra_8_4(self, cfg):
                 else:
                     self.insert_in_dict(extern_var_without_def, tok.str, tok)
             else:
-                 self.insert_in_dict(extern_var_without_def, var.nameToken.str, var.nameToken)
+                self.insert_in_dict(extern_var_without_def, var.nameToken.str, var.nameToken)
 
         for var in extern_var_with_def:
             if var not in extern_var_without_def:

addons/namingng.py

@@ -200,8 +200,8 @@ def check_include_guards(conf,cfg,unguarded_include_files):
     # - test whether include guards are in place
     max_linenr = conf.include_guard.get('max_linenr', 5)
 
-    def report(directive,msg,errorId,column=0):
-        reportNamingError(directive,msg,errorId,column=column)
+    def report(directive,msg,errorId,severity='style',column=0):
+        reportNamingError(directive,msg,errorId,severity=severity,column=column)
 
     def report_pending_ifndef(directive,column):
         report(directive,'include guard #ifndef is not followed by #define','includeGuardIncomplete',column=column)
@@ -266,8 +266,10 @@ def report_pending_ifndef(directive,column):
 def process(dumpfiles, configfile):
     conf = loadConfig(configfile)
 
+    # pylint: disable-next=no-member - TODO: fix this
     if conf.include_guard:
         global include_guard_header_re
+        # pylint: disable-next=no-member - TODO: fix this
         include_guard_header_re = conf.include_guard.get('RE_HEADERFILE',"[^/].*\\.h\\Z")
 
     for afile in dumpfiles:

htmlreport/cppcheck-htmlreport

@@ -422,6 +422,7 @@ def git_blame(errors, path, file, blame_options):
     full_path = os.path.join(path, file)
     path, filename = os.path.split(full_path)
 
+    cwd = os.getcwd()
     if path:
         os.chdir(path)