Skip to content
Snippets Groups Projects
Commit 1a680ccd authored by Pierre-Yves David's avatar Pierre-Yves David
Browse files

extract: assign message only once

This return is shared between the two cases. We can move it earlier.
parent 25a58881
No related branches found
No related tags found
No related merge requests found
...@@ -262,6 +262,7 @@ ...@@ -262,6 +262,7 @@
if subject and not message.startswith(subject): if subject and not message.startswith(subject):
message = '%s\n%s' % (subject, message) message = '%s\n%s' % (subject, message)
data['message'] = message
tmpfp.close() tmpfp.close()
if not diffs_seen: if not diffs_seen:
os.unlink(tmpname) os.unlink(tmpname)
...@@ -265,7 +266,6 @@ ...@@ -265,7 +266,6 @@
tmpfp.close() tmpfp.close()
if not diffs_seen: if not diffs_seen:
os.unlink(tmpname) os.unlink(tmpname)
data['message'] = message
data['user'] = user data['user'] = user
data['date'] = date data['date'] = date
data['branch'] = branch data['branch'] = branch
...@@ -278,7 +278,6 @@ ...@@ -278,7 +278,6 @@
p2 = parents.pop(0) p2 = parents.pop(0)
data['filename'] = tmpname data['filename'] = tmpname
data['message'] = message
data['user'] = user data['user'] = user
data['date'] = date data['date'] = date
data['branch'] = branch data['branch'] = branch
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment