patchbomb: fix up confusion between strings and lists of strings.
authorbos@serpentine.internal.keyresearch.com
Mon, 29 Aug 2005 12:15:37 -0700
changeset 1154 c3cb9f39a91f
parent 1153 fa9ae7df88a9
child 1155 92697ad28f08
child 1159 b6f5a947e62e
patchbomb: fix up confusion between strings and lists of strings.
contrib/patchbomb
--- a/contrib/patchbomb	Mon Aug 29 20:37:07 2005 +0200
+++ b/contrib/patchbomb	Mon Aug 29 12:15:37 2005 -0700
@@ -172,11 +172,13 @@
         len(patches),
         opts['subject'] or
         prompt('Subject:', rest = ' [PATCH 0 of %d] ' % len(patches)))
-    to = opts['to'] or ui.config('patchbomb', 'to') or prompt('To')
-    to = [t.strip() for t in to.split(',')]
-    cc = (opts['cc'] or ui.config('patchbomb', 'cc') or
-          prompt('Cc', default = ''))
-    cc = (cc and [c.strip() for c in cc.split(',')]) or []
+
+    def getaddrs(opt, prpt, default = None):
+        addrs = opts[opt] or (ui.config('patchbomb', opt) or
+                              prompt(prpt, default = default)).split(',')
+        return [a.strip() for a in addrs if a.strip()]
+    to = getaddrs('to', 'To')
+    cc = getaddrs('cc', 'Cc', '')
 
     ui.write('Finish with ^D or a dot on a line by itself.\n\n')