commit 1d947b30342163b723c96db563967323535fef45
author David Pursehouse <david.pursehouse@sonymobile.com> Thu Oct 25 12:23:11 2012 +0900
committer David Pursehouse <david.pursehouse@sonymobile.com> Tue Oct 30 10:28:20 2012 +0900
tree b6b02d02df01792b356b9e50ceeaadcaf3a2e8c5
parent 2d113f35460051823ea54d61c5c939565518f969

Even more coding style cleanup

Fixing some more pylint warnings:

W1401: Anomalous backslash in string
W0623: Redefining name 'name' from outer scope
W0702: No exception type(s) specified
E0102: name: function already defined line n

Change-Id: I5afcdb4771ce210390a79981937806e30900a93c

git_command.py

diff --git a/git_command.py b/git_command.py
index a40e6c0..39f795f 100644
--- a/git_command.py
+++ b/git_command.py
@@ -132,15 +132,15 @@
                gitdir = None):
     env = os.environ.copy()
 
-    for e in [REPO_TRACE,
+    for key in [REPO_TRACE,
               GIT_DIR,
               'GIT_ALTERNATE_OBJECT_DIRECTORIES',
               'GIT_OBJECT_DIRECTORY',
               'GIT_WORK_TREE',
               'GIT_GRAFT_FILE',
               'GIT_INDEX_FILE']:
-      if e in env:
-        del env[e]
+      if key in env:
+        del env[key]
 
     if disable_editor:
       _setenv(env, 'GIT_EDITOR', ':')