Re: PATCH: vux/vuxctl completion

[Clint Adams <clint@xxxxxxx>]

> Since in 4.1, matches are grouped where they have the same description,
> it is much nicer to give the description to both the long and short
> option. The convention is to do this with brace expansion:
> 
>   '(-r --report)'{-r,--report}'[report]' \
>   '(-c --report-cycles)'{-c,--report-cycles}'[report cycles]:count' \
> 
> Except the descriptions could be rather more, um, descriptive.

Index: Completion/Unix/Command/_mtr
===================================================================
RCS file: /cvsroot/zsh/zsh/Completion/Unix/Command/_mtr,v
retrieving revision 1.1
diff -u -r1.1 _mtr
--- Completion/Unix/Command/_mtr	11 Mar 2004 21:58:49 -0000	1.1
+++ Completion/Unix/Command/_mtr	12 Mar 2004 15:30:08 -0000
@@ -1,28 +1,16 @@
 #compdef mtr
 
 _arguments -C \
-	'(--help)-h[help]' \
-	'(-h)--help' \
-	'(--version)-v[version]' \
-	'(-v)--version' \
-	'(--report)-r[report]' \
-	'(-r)--report' \
-	'(--report--cycles)-c[report cycles]:count:' \
-	'(-c)--report-cycles:count:' \
-	'(--psize)-p[packet size]:packet size in bytes:' \
-	'(-p)--psize:packet size in bytes:' \
-	'(--curses)-t[curses]' \
-	'(-t)--curses' \
-	'(--no-dns)-n[no dns]' \
-	'(-n)--no-dns' \
-	'(--gtk)-g[gtk]' \
-	'(-g)--gtk' \
-	'(--split)-s[split]' \
-	'(-s)--split' \
-	'(--raw)-l[raw]' \
-	'(-l)--raw' \
-	'(--address)-a[source IP]:source IP:' \
-	'(-a)--address:source IP:' \
-	'(--interval)-i[interval]:seconds:' \
-	'(-i)--interval:seconds:' \
-	':hostname:_hosts'
+	'(-h --help)'{-h,--help}'[help]' \
+	'(-v --version)'{-v,--version}'[version]' \
+	'(-r --report)'{-r,--report}'[report mode]' \
+	'(-c --report-cycles)'{-c,--report-cycles}'[report cycles]:number of pings' \
+	'(-p --psize)'{-p,--psize}'[packet size]:number of bytes' \
+	'(-t --curses)'{-t,--curses}'[curses UI]' \
+	'(-n --no-dns)'{-n,--no-dns}'[no DNS lookups]' \
+	'(-g --gtk)'{-g,--gtk}'[GTK+ UI]' \
+	'(-s --split)'{-s,--split}'[output for a split-user interface]' \
+	'(-l --raw)'{-l,--raw}'[raw output format]' \
+	'(-a --address)'{-a,--address}'[source address]:source IP:_hosts' \
+	'(-i --interval)'{-i,--interval}'[ping interval]:number of seconds' \
+	':destination host:_hosts'