[PATCH] contib/nmbug/nmbug-status: leftover whitespaces, indentation & quoting

Tomi Ollila tomi.ollila at iki.fi
Tue Jul 10 15:29:11 PDT 2012


The initial nmbug-status was pretty consistent in it's whitespacing
but a few lines had some leftover slips. Those are now "corrected".

Also, most of the code used ' as quoting char. As in Python one can
use ' and " interchangeably some code used " instead of '. However
the usage of those were inconsistent. Now all quotes that python
parses are ':s (only quoted content uses ":s).

No functional changes.
---

I tested that this produces identical output compared to the unpatched version.

 contrib/nmbug/nmbug-status |   41 ++++++++++++++++++++---------------------
 1 files changed, 20 insertions(+), 21 deletions(-)

diff --git a/contrib/nmbug/nmbug-status b/contrib/nmbug/nmbug-status
index 6aa86a0..2abb3e4 100755
--- a/contrib/nmbug/nmbug-status
+++ b/contrib/nmbug/nmbug-status
@@ -18,10 +18,10 @@ import subprocess
 # parse command line arguments
 
 parser = argparse.ArgumentParser()
-parser.add_argument("--text", help="output plain text format",
-                    action="store_true")
+parser.add_argument('--text', help='output plain text format',
+                    action='store_true')
 
-parser.add_argument("--config", help="load config from given file")
+parser.add_argument('--config', help='load config from given file')
 
 
 args = parser.parse_args()
@@ -31,18 +31,18 @@ args = parser.parse_args()
 if args.config != None:
     fp = open(args.config)
 else:
-    nmbhome = os.getenv('NMBGIT', os.path.expanduser("~/.nmbug"))
+    nmbhome = os.getenv('NMBGIT', os.path.expanduser('~/.nmbug'))
 
     # read only the first line from the pipe
-    sha1 =  subprocess.Popen(['git', '--git-dir', nmbhome,
-                     'show-ref', '-s', 'config'],
-                      stdout=subprocess.PIPE).stdout.readline()
+    sha1 = subprocess.Popen(['git', '--git-dir', nmbhome,
+                             'show-ref', '-s', 'config'],
+                            stdout=subprocess.PIPE).stdout.readline()
 
     sha1 = sha1.rstrip()
 
-    fp =  subprocess.Popen(['git', '--git-dir', nmbhome,
-                            'cat-file', 'blob', sha1+':status-config.json'],
-                           stdout=subprocess.PIPE).stdout
+    fp = subprocess.Popen(['git', '--git-dir', nmbhome,
+                           'cat-file', 'blob', sha1+':status-config.json'],
+                          stdout=subprocess.PIPE).stdout
 
 config = json.load(fp)
 
@@ -60,11 +60,10 @@ def clear_last():
 
 def print_view(title, query, comment):
 
-    query_string = " and ".join(query)
+    query_string = ' and '.join(query)
     q_new = notmuch.Query(db, query_string)
     q_new.set_sort(notmuch.Query.SORT.OLDEST_FIRST)
 
-
     last['thread_id'] = ''
 
     if output_format == 'html':
@@ -94,7 +93,7 @@ def print_view(title, query, comment):
                 val = rfc822.parseaddr(val)[0]
 
             if last[header] == val:
-                out[header] = ""
+                out[header] = ''
             else:
                 out[header] = val.encode('utf-8')
                 last[header] = val
@@ -109,14 +108,14 @@ def print_view(title, query, comment):
             else:
                 br = ''
                 out['subject'] = '<a href="http://mid.gmane.org/%s">%s</a>' \
-                                        % (urllib.quote(mid), out['subject'])
-
-            print " <tr><td>%s %s" % (br, out['date'])
-            print "</td><td>%s %s" % (br, out['id'])
-            print "</td></tr>"
-            print " <tr><td>%s" % out['from']
-            print "</td><td>%s" % out['subject']
-            print "</td></tr>\n"
+                    % (urllib.quote(mid), out['subject'])
+
+            print ' <tr><td>%s %s' % (br, out['date'])
+            print '</td><td>%s %s' % (br, out['id'])
+            print '</td></tr>'
+            print ' <tr><td>%s' % out['from']
+            print '</td><td>%s' % out['subject']
+            print '</td></tr>\n'
         else:
             print '%(date)-10.10s %(from)-20.20s %(subject)-40.40s\n%(id)72s\n' % out
 
-- 
1.7.1



More information about the notmuch mailing list