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 d98ed09f84..accb1b0f1d 100755 --- a/git-p4.py +++ b/git-p4.py @@ -3579,7 +3579,6 @@ def getBranchMapping(self): if source not in self.knownBranches: lostAndFoundBranches.add(source) - for branch in lostAndFoundBranches: self.knownBranches[branch] = branch @@ -3803,7 +3802,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 @@ -3831,7 +3829,6 @@ def importHeadRevision(self, revision): print("IO error details: {}".format(err)) print(self.gitError.read()) - def importRevisions(self, args, branch_arg_given): changes = [] -- 2.34.1