cleanup

- use re.sub() iso. if for stripping ...
- spacing nits

Signed-off-by: Han-Wen Nienhuys <hanwen@google.com>

diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4
index 05b308f5be19f1aad5bf45b4215a6bdbc5eb54ec..8d649dd7628fd9c8079549a9bb73548255512481 100755 (executable)
--- a/contrib/fast-import/git-p4
+++ b/contrib/fast-import/git-p4
@@ -16,39 +16,39 @@ from sets import Set;
 gitdir = os.environ.get("GIT_DIR", "")
 silent = False
 
 gitdir = os.environ.get("GIT_DIR", "")
 silent = False
 

-def write_pipe (c, str):
+def write_pipe(c, str):

     if not silent:
     if not silent:

-        sys.stderr.write ('writing pipe: %s\n' % c)
+        sys.stderr.write('writing pipe: %s\n' % c)

 
     ## todo: check return status
 
     ## todo: check return status

-    pipe = os.popen (c, 'w')
+    pipe = os.popen(c, 'w')

     val = pipe.write(str)
     val = pipe.write(str)

-    if pipe.close ():
-        sys.stderr.write ('Command failed')
-        sys.exit (1)
+    if pipe.close():
+        sys.stderr.write('Command failed')
+        sys.exit(1)

 
     return val
 
 
     return val
 

-def read_pipe (c):
-    sys.stderr.write ('reading pipe: %s\n' % c)
+def read_pipe(c):
+    sys.stderr.write('reading pipe: %s\n' % c)

     ## todo: check return status
     ## todo: check return status

-    pipe = os.popen (c, 'rb')
+    pipe = os.popen(c, 'rb')

     val = pipe.read()
     val = pipe.read()

-    if pipe.close ():
-        sys.stderr.write ('Command failed')
-        sys.exit (1)
+    if pipe.close():
+        sys.stderr.write('Command failed')
+        sys.exit(1)

 
     return val
 
 
 
     return val
 
 

-def read_pipe_lines (c):
-    sys.stderr.write ('reading pipe: %s\n' % c)
+def read_pipe_lines(c):
+    sys.stderr.write('reading pipe: %s\n' % c)

     ## todo: check return status
     ## todo: check return status

-    pipe = os.popen (c, 'rb')
+    pipe = os.popen(c, 'rb')

     val = pipe.readlines()
     val = pipe.readlines()

-    if pipe.close ():
-        sys.stderr.write ('Command failed')
-        sys.exit (1)
+    if pipe.close():
+        sys.stderr.write('Command failed')
+        sys.exit(1)

 
     return val
 
 
     return val
 


@@ -986,9 +986,7 @@ class P4Sync(Command):
         elif len(self.previousDepotPath) == 0:
             self.revision = "#head"
 
         elif len(self.previousDepotPath) == 0:
             self.revision = "#head"
 

-        if self.depotPath.endswith("..."):
-            self.depotPath = self.depotPath[:-3]
-
+        self.depotPath = re.sub ("\.\.\.$", "", self.depotPath)

         if not self.depotPath.endswith("/"):
             self.depotPath += "/"
 
         if not self.depotPath.endswith("/"):
             self.depotPath += "/"