fix Loop variable used outside the loop

Author: imkaka

git/diff.py

@@ -415,13 +415,14 @@ def _index_from_patch_format(cls, repo, proc):
         text = b''.join(text)
         index = DiffIndex()
         previous_header = None
-        for header in cls.re_header.finditer(text):
+        header = None
+        for _header in cls.re_header.finditer(text):
             a_path_fallback, b_path_fallback, \
                 old_mode, new_mode, \
                 rename_from, rename_to, \
                 new_file_mode, deleted_file_mode, \
                 a_blob_id, b_blob_id, b_mode, \
-                a_path, b_path = header.groups()
+                a_path, b_path = _header.groups()
 
             new_file, deleted_file = bool(new_file_mode), bool(deleted_file_mode)
 
@@ -431,7 +432,7 @@ def _index_from_patch_format(cls, repo, proc):
             # Our only means to find the actual text is to see what has not been matched by our regex,
             # and then retro-actively assign it to our index
             if previous_header is not None:
-                index[-1].diff = text[previous_header.end():header.start()]
+                index[-1].diff = text[previous_header.end():_header.start()]
             # end assign actual diff
 
             # Make sure the mode is set if the path is set. Otherwise the resulting blob is invalid
@@ -450,7 +451,8 @@ def _index_from_patch_format(cls, repo, proc):
                               rename_to,
                               None, None, None))
 
-            previous_header = header
+            previous_header = _header
+            header = _header
         # end for each header we parse
         if index:
             index[-1].diff = text[header.end():]

git/objects/submodule/base.py

@@ -846,7 +846,7 @@ def remove(self, module=True, force=False, configuration=True, dry_run=False):
                     # have to manually delete references as python's scoping is
                     # not existing, they could keep handles open ( on windows this is a problem )
                     if len(rrefs):
-                        del(rref)
+                        del(rref) # skipcq:PYL-W0631
                     # END handle remotes
                     del(rrefs)
                     del(remote)

git/refs/log.py

@@ -228,7 +228,7 @@ def entry_at(cls, filepath, index):
                 # END abort on eof
             # END handle runup
 
-            if i != index or not line:
+            if i != index or not line: # skipcq:PYL-W0631
                 raise IndexError
             # END handle exception
 

git/util.py

@@ -133,7 +133,7 @@ def join_path(a, *p):
     '/' instead of possibly '\' on windows."""
     path = a
     for b in p:
-        if len(b) == 0:
+        if not b:
             continue
         if b.startswith('/'):
             path += b[1:]
@@ -386,7 +386,7 @@ def _parse_progress_line(self, line):
         # Compressing objects: 100% (2/2)
         # Compressing objects: 100% (2/2), done.
         self._cur_line = line = line.decode('utf-8') if isinstance(line, bytes) else line
-        if len(self.error_lines) > 0 or self._cur_line.startswith(('error:', 'fatal:')):
+        if self.error_lines or self._cur_line.startswith(('error:', 'fatal:')):
             self.error_lines.append(self._cur_line)
             return