patchbomb: fix up confusion between strings and lists of strings.
--- 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')