Re: Small bug in zsh 2.6beta9

["Zvi Har'El" <rl@xxxxxxxxxxxxxxxxxxxx>]

Peter,

YOU were right all the time. The two lines of code are identical. Perhaps
the idiom of checking if(a) rather then if(a!=0) is not clear to people
who are not seasoned C programmers, but personally I prefer your original code.


On Mon Jun  5 12:37:34 1995, P.Stephenson@xxxxxxxxxxxxx wrote about ``Re: Small bug in zsh 2.6beta9'':
> 
> heiko@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx wrote:
> > In the file exec.c, line 493, the line
> > 
> >     if (ornext ^ ret) {
> > 
> > should read
> > 
> >     if (ornext ^ (ret!=0)) {
> 
> You're quite right --- this is my fault again.  Here's an actual patch
> for that.
> 
> *** Src/exec.c.ret	Mon Jun  5 10:16:14 1995
> --- Src/exec.c	Mon Jun  5 10:34:30 1995
> ***************
> *** 506,512 ****
>   	     */
>   	    ornext = (slist->type == ORNEXT);
>   
> ! 	    if (ornext ^ ret) {
>   		/*
>   		 * If either (1) it's an or, and the return code
>   		 * was zero, or (2) it's an and, and the return
> --- 506,512 ----
>   	     */
>   	    ornext = (slist->type == ORNEXT);
>   
> ! 	    if (ornext ^ (ret != 0)) {
>   		/*
>   		 * If either (1) it's an or, and the return code
>   		 * was zero, or (2) it's an and, and the return
> 
> -- 
> Peter Stephenson <P.Stephenson@xxxxxxxxxxxxx>  Tel: +44 1792 205678 extn. 4461
> WWW:  http://python.swan.ac.uk/~pypeters/      Fax: +44 1792 295324
> Department of Physics, University of Wales, Swansea,
> Singleton Park, Swansea, SA2 8PP, U.K.
> 
> 
> 


-- 
Dr. Zvi Har'El <rl@xxxxxxxxxxxxxxxxxxx>              Department of Mathematics
+972-4-294094(Phone)                 Technion - Israel Institute of Technology
+972-4-324654(FAX)     http://gauss.technion.ac.il/~rl     Haifa 32000, ISRAEL
``If you can't say somethin' nice, don't say nothin' at all.''--Thumper (1942)