Re: crash in the cvs head build of Feb 13 08:40

[Sam Steingold <sds@gnu.org>]

> * Richard M. Stallman <ezf@tah.bet> [2006-02-19 12:45:43 -0500]:
>
> In this case I think it is better to be less defensive
> and stick more clearly to the rules from which the code was written.
> Does this patch fix the bug?

I cannot reproduce the bug (with or without the patch).

> *** xmenu.c	07 Feb 2006 18:13:41 -0500	1.299
> --- xmenu.c	19 Feb 2006 11:46:11 -0500	
> ***************
> *** 1807,1814 ****
>   	      wv->enabled = 1;
>   	      wv->button_type = BUTTON_TYPE_NONE;
>   	      wv->help = Qnil;
>   	    }
> ! 	  save_wv = wv;
>   	  prev_wv = 0;
>   	  i += MENU_ITEMS_PANE_LENGTH;
>   	}
> --- 1807,1817 ----
>   	      wv->enabled = 1;
>   	      wv->button_type = BUTTON_TYPE_NONE;
>   	      wv->help = Qnil;
> + 	      save_wv = wv;
>   	    }
> ! 	  else
> ! 	    save_wv = first_wv;
> ! 
>   	  prev_wv = 0;
>   	  i += MENU_ITEMS_PANE_LENGTH;
>   	}

-- 
Sam Steingold (http://www.podval.org/~sds) running FC3 GNU/Linux
http://truepeace.org http://www.dhimmi.com http://www.jihadwatch.org
http://www.iris.org.il http://www.camera.org http://pmw.org.il http://ffii.org
The software said it requires Windows 3.1 or better, so I installed Linux.