Re: zsh-3.1.0-test3 release

["Bart Schaefer" <schaefer@candle.brasslantern.com>]

On Nov 17,  5:27am, Zoltan Hidvegi wrote:
} Subject: zsh-3.1.0-test3 release
}
} I just made a new test release:
} 
}     41654 Nov 17 05:21 zsh-3.1.0-test3.diff.gz

The change in zle_tricky.c to handle leading tildes in spell-word is
badly broken.  You can't just declare char **x; and then assign to *x.

*** Src/zle_tricky.c.0	Sun Nov 17 15:04:25 1996
--- Src/zle_tricky.c	Tue Nov 19 02:17:37 1996
***************
*** 674,680 ****
  	    inwhat = IN_CMD;
  
  	if (lst == COMP_SPELL) {
! 	    char **x, *q;
  
  	    for (q = s; *q; q++)
  		if (INULL(*q))
--- 674,680 ----
  	    inwhat = IN_CMD;
  
  	if (lst == COMP_SPELL) {
! 	    char *x, *q;
  
  	    for (q = s; *q; q++)
  		if (INULL(*q))
***************
*** 682,694 ****
  	    cs = wb;
  	    foredel(we - wb);
  	    HEAPALLOC {
! 		untokenize(*x = dupstring(s));
  		if (*s == Tilde || *s == Equals || *s == String)
! 		    **x = *s;
! 		spckword(x, 0, lincmd, 0);
  	    } LASTALLOC;
! 	    untokenize(*x);
! 	    inststr(*x);
  	} else if (COMP_ISEXPAND(lst)) {
  	    /* Do expansion. */
  	    char *ol = (olst == COMP_EXPAND_COMPLETE) ?
--- 682,694 ----
  	    cs = wb;
  	    foredel(we - wb);
  	    HEAPALLOC {
! 		untokenize(x = dupstring(s));
  		if (*s == Tilde || *s == Equals || *s == String)
! 		    *x = *s;
! 		spckword(&x, 0, lincmd, 0);
  	    } LASTALLOC;
! 	    untokenize(x);
! 	    inststr(x);
  	} else if (COMP_ISEXPAND(lst)) {
  	    /* Do expansion. */
  	    char *ol = (olst == COMP_EXPAND_COMPLETE) ?

-- 
Bart Schaefer                             Brass Lantern Enterprises
http://www.well.com/user/barts            http://www.nbn.com/people/lantern