Re: More crash Re: More incmdpos stuff Re: Crash Re: Trial for 5.0.7

[Peter Stephenson <p.stephenson@xxxxxxxxxxx>]

On Fri, 03 Oct 2014 08:33:27 -0700
Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx> wrote:
> On Oct 3,  4:15pm, Peter Stephenson wrote:
> } Subject: Re: More crash Re: More incmdpos stuff Re: Crash Re: Trial for 5.
> }
> } The explanation is I screwed up.  *complex should never be set to 0
> } except where the variable pointed to is created --- it should only be
> } conditionally set to 1.  (For zsh-shaped values of "never".)
> 
> Does that mean we need this, too?
> 
> 
> --- a/Src/parse.c
> +++ b/Src/parse.c
> @@ -1517,7 +1517,8 @@ par_funcdef(int *complex)
>  	    num++;
>  	    zshlex();
>  	}
> -	*cmplx = (num > 0);
> +	if (num > 0)
> +	    *cmplx = 1;
>  	ecbuf[parg] = ecused - parg; /*?*/
>  	ecbuf[parg+1] = num;
>      }

Yes, that's yet another variant syntax SNAFU. Thanks.

pws