Remove debugging code and unused 'unescape()' function.

Author: helmesjo

compiledb/parser.py

@@ -84,7 +84,6 @@ def skip_line(cmd, reason):
 
     # Process build log
     for line in build_log:
-        # logger.debug("-------------> Line: {}".format(line))
         lineno += 1
         # Concatenate line if need
         accumulate_line = line
@@ -118,13 +117,9 @@ def skip_line(cmd, reason):
         if not commands:
             result.skipped += 1
 
-        # logger.debug("-------------> Commands: {}".format(commands))
-
         for c in commands:
             filepath = c['filepath']
-            # logger.debug("-------------> Filepath: {}".format(filepath))
             cmd = c['cmd']
-            # logger.debug("-------------> Command: {}".format(cmd))
             if filepath is None:
                 skip_line(cmd, 'Empty file name')
                 continue
@@ -143,7 +138,7 @@ def skip_line(cmd, reason):
 
             # add entry to database
             tokens = c['tokens']
-            arguments = [unescape(a) for a in tokens[len(wrappers):]]
+            arguments = [a for a in tokens[len(wrappers):]]
 
             compiler = get_compiler(arguments[0])
 
@@ -192,8 +187,9 @@ def cross_platform_argline(s, platform='this'):
               1 = POSIX; 
               0 = Windows/CMD
               (other values reserved)
+
+    Cudos: https://stackoverflow.com/a/35900070/2349761
     """
-    logger.debug("-------------> Platform: {} - {}".format(platform, sys.platform))
     if platform == 'this':
         platform = (sys.platform != 'win32')
     if platform == 1:
@@ -205,9 +201,7 @@ def cross_platform_argline(s, platform='this'):
 
     args = []
     accu = None   # collects pieces of one arg
-    # logger.debug("-------------> String: {}".format(s))
     for qs, qss, esc, pipe, word, white, fail in re.findall(RE_CMD_LEX, s):
-        # logger.debug("-------------> Regex: qs:{} qss:{} esc:{} pipe:{} word:{} white:{} fail:{}".format(qs, qss, esc, pipe, word, white, fail))
         if word:
             if platform == 0:
                 word = word.replace('\\\\', '\\').replace('\\', '\\\\')
@@ -223,10 +217,8 @@ def cross_platform_argline(s, platform='this'):
         elif fail:
             raise ValueError("invalid or incomplete shell string")
         elif qs:
-            # word = qs.replace('\\"', '"').replace('\\\\', '\\')
             if platform == 0:
                 word = word.replace('""', '"')
-                # word = word.replace('\\\\', '\\').replace('\\', '\\\\')
                 word = repr(qs).encode().decode('unicode_escape')
         else:
             word = qss   # may be even empty; must be last
@@ -243,10 +235,7 @@ class CommandProcessor(bashlex.ast.nodevisitor):
        looking for and extracting compilation commands."""
     def process(line, wd):
         args = cross_platform_argline(line)
-        # for arg in args:
-        #     logger.debug("-------------> Arg: {}".format(arg))
         line = " ".join(args)
-        # logger.debug("-------------> Argline: {}".format(line))
         trees = bashlex.parser.parse(line)
         if not trees:
             return []
@@ -303,7 +292,6 @@ def visitword(self, node, word):
                 self.compiler = word
             else:
                 self.wrappers.append(word)
-                logger.debug('Not a compiler: {}'.format(word))
         elif (file_regex.match(word)):
             self.filepath = word
 
@@ -318,8 +306,4 @@ def check_last_cmd(self):
         # reset state to process new command
         self.reset()
 
-
-def unescape(s):
-    return s
-
 # ex: ts=2 sw=4 et filetype=python