From bce4c5fc0b6aa9e599fc181c5350b461dafecee2 Mon Sep 17 00:00:00 2001
From: Han-Wen Nienhuys <hanwen@google.com>
Date: Wed, 23 May 2007 17:14:33 -0300
Subject: [PATCH] cleanup

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

Signed-off-by: Han-Wen Nienhuys <hanwen@google.com>
---
 contrib/fast-import/git-p4 | 40 ++++++++++++++++++--------------------
 1 file changed, 19 insertions(+), 21 deletions(-)

diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4
index 05b308f5be..8d649dd762 100755
--- 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
 
-def write_pipe (c, str):
+def write_pipe(c, str):
     if not silent:
-        sys.stderr.write ('writing pipe: %s\n' % c)
+        sys.stderr.write('writing pipe: %s\n' % c)
 
     ## todo: check return status
-    pipe = os.popen (c, 'w')
+    pipe = os.popen(c, 'w')
     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
 
-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
-    pipe = os.popen (c, 'rb')
+    pipe = os.popen(c, 'rb')
     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
 
 
-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
-    pipe = os.popen (c, 'rb')
+    pipe = os.popen(c, 'rb')
     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
 
@@ -986,9 +986,7 @@ class P4Sync(Command):
         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 += "/"