something wrong with _arguments wordsplitting

something wrong with _arguments wordsplitting

[Andrej Borsenkow]

SOCKS5 v1.0r10:

bor@itsrm2% CFLAGS=-O ./configure --prefix=/tools --help    
Usage: configure [options] [host]
Options: [defaults in brackets after descriptions]
   ...
--with-static           Turn on static compiling
   ...
bor@itsrm2% CFLAGS=-O ./configure --prefix=/tools --with-st<TAB>
bor@itsrm2% CFLAGS=-O ./configure --prefix=/tools --with-staticTurn

Oops! Hmm, may be, there are TABs in between?        

-andrej

Have a nice DOS!
B >> 

PATCH: Re: something wrong with _arguments wordsplitting

[Bart Schaefer]

On May 6,  1:47pm, Andrej Borsenkow wrote:
} Subject: something wrong with _arguments wordsplitting
}
} bor@itsrm2% CFLAGS=-O ./configure --prefix=/tools --with-st<TAB>
} bor@itsrm2% CFLAGS=-O ./configure --prefix=/tools --with-staticTurn
} 
} Oops! Hmm, may be, there are TABs in between?        

That does appear to have been it.  This should fix it.

The comma wasn't necessary in the final pattern any more, was it?  All
the commas should be zapped by then.

Index: Completion/Base/_arguments
===================================================================
@@ -60,16 +60,16 @@
     # The parameter expansion trickery first gets the lines as separate
     # array elements. Then we select all lines whose first non-blank
     # character is a hyphen. Since some commands document more than one
-    # option per line, separated by commas, we convert commas int
+    # option per line, separated by commas, we convert commas into
     # newlines and then split the result again at newlines after joining 
     # the old array elements with newlines between them. Then we select
     # those elements that start with two hyphens, remove anything up to
-    # those hyphens and anything from the space or comma after the
-    # option up to the end. 
+    # those hyphens and anything from the space or tab after the
+    # option up to the end.
 
     lopts=("--${(@)^${(@)${(@)${(@M)${(@ps:\n:j:\n:)${(@)${(@M)${(@f)$(_call options ${~words[1]} --help 2>&1)//\[--/
 --}:#[ 	]#-*}//,/
-}}:#[ 	]#--*}#*--}%%[], ]*}:#}")
+}}:#[ 	]#--*}#*--}%%[]	 ]*}:#}")
     lopts=( "${(@)lopts:#--}" )
 
     # Now remove all ignored options ...

-- 
Bart Schaefer                                 Brass Lantern Enterprises
http://www.well.com/user/barts              http://www.brasslantern.com