PEP8 recommends that blank lines should be used sparingly to separate sections in the "Blank Lines" section: https://www.python.org/dev/peps/pep-0008/#blank-lines This patch replaces all double blank-line separations with a single blank line. Signed-off-by: Joel Holdsworth <jholdsworth@xxxxxxxxxx> --- git-p4.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/git-p4.py b/git-p4.py index 2f2450f7d9..c003b6246d 100755 --- a/git-p4.py +++ b/git-p4.py @@ -3521,7 +3521,6 @@ def getBranchMapping(self): if source not in self.knownBranches: lostAndFoundBranches.add(source) - for branch in lostAndFoundBranches: self.knownBranches[branch] = branch @@ -3745,7 +3744,6 @@ def importHeadRevision(self, revision): sys.stderr.write("p4 exitcode: %s\n" % info['p4ExitCode']) sys.exit(1) - change = int(info["change"]) if change > newestRevision: newestRevision = change @@ -3773,7 +3771,6 @@ def importHeadRevision(self, revision): print("IO error details: {}".format(err)) print(self.gitError.read()) - def importRevisions(self, args, branch_arg_given): changes = [] -- 2.35.GIT