[PATCH 04/14] git p4: inline listExistingP4GitBranches
- From
- Pete Wyckoff <pw@padd.com>
- Date
- Jan 15, 2013, 00:46 UTC
- Message-ID
- <1358210828-2369-5-git-send-email-pw@padd.com>
- In-Reply-To
- <1358210828-2369-1-git-send-email-pw@padd.com>
It is four lines of code used in only one place. Simplify by including it where it is used.
Signed-off-by: Pete Wyckoff <pw@padd.com> --- git-p4.py | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/git-p4.py b/git-p4.py index 03680b0..8814049 100755 --- a/git-p4.py +++ b/git-p4.py @@ -2518,13 +2518,6 @@ class P4Sync(Command, P4UserMap): branch = branch[len(self.projectName):] self.knownBranches[branch] = branch - def listExistingP4GitBranches(self): - # branches holds mapping from name to commit - branches = p4BranchesInGit(self.importIntoRemotes) - self.p4BranchesInGit = branches.keys() - for branch in branches.keys(): - self.initialParents[self.refPrefix + branch] = branches[branch] - def updateOptionDict(self, d): option_keys = {} if self.keepRepoPath: @@ -2805,7 +2798,12 @@ class P4Sync(Command, P4UserMap): if args == []: if self.hasOrigin: createOrUpdateBranchesFromOrigin(self.refPrefix, self.silent) - self.listExistingP4GitBranches() + + # branches holds mapping from branch name to sha1 + branches = p4BranchesInGit(self.importIntoRemotes) + self.p4BranchesInGit = branches.keys() + for branch in branches.keys(): + self.initialParents[self.refPrefix + branch] = branches[branch] if len(self.p4BranchesInGit) > 1: if not self.silent:
-- 1.8.1.350.gdbf6fd0