Merge lp:~zyga/checkbox/fix-1316492 into lp:checkbox

Proposed by Zygmunt Krynicki
Status: Merged
Approved by: Brendan Donegan
Approved revision: 2988
Merged at revision: 2992
Proposed branch: lp:~zyga/checkbox/fix-1316492
Merge into: lp:checkbox
Diff against target: 27 lines (+3/-3)
1 file modified
plainbox/plainbox/impl/secure/rfc822.py (+3/-3)
To merge this branch: bzr merge lp:~zyga/checkbox/fix-1316492
Reviewer Review Type Date Requested Status
Brendan Donegan (community) Approve
Review via email: mp+219299@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

Yes, of course - format across the board

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plainbox/plainbox/impl/secure/rfc822.py'
2--- plainbox/plainbox/impl/secure/rfc822.py 2014-03-23 20:28:58 +0000
3+++ plainbox/plainbox/impl/secure/rfc822.py 2014-05-13 08:03:02 +0000
4@@ -318,14 +318,14 @@
5 Dump this record to a stream
6 """
7 def _dump_part(stream, key, values):
8- stream.write("%s:\n" % key)
9+ stream.write("{}:\n".format(key))
10 for value in values:
11 if not value:
12 stream.write(" .\n")
13 elif value == ".":
14 stream.write(" ..\n")
15 else:
16- stream.write(" %s\n" % value)
17+ stream.write(" {}\n".format(value))
18 for key, value in self.data.items():
19 if isinstance(value, (list, tuple)):
20 _dump_part(stream, key, value)
21@@ -335,7 +335,7 @@
22 values = values[:-1]
23 _dump_part(stream, key, values)
24 else:
25- stream.write("%s: %s\n" % (key, value))
26+ stream.write("{}: {}\n".format(key, value))
27 stream.write("\n")
28
29

Subscribers

People subscribed via source and target branches