Subject: FVWM: fvwm-2.0.38 patch to FvwmButtons: Exec "" ...

From: Grant McDorman <[email-masked]>
Date: Thu, 2 Nov 1995 18:12:00 -0500

>From the ChangeLog in 2.0.38:

	6. FvwmButtons now handles Exec "" foobar correctly.

We have not found this to be the case. In fact,
	Exec foobar
was also broken, and only
	Exec "name" foobar
seemed to work.

The attached patch corrects the problem.

*** FvwmButtons.c.orig	Tue Oct 17 10:20:39 1995
--- FvwmButtons.c	Thu Nov  2 18:08:03 1995
***************
*** 363,369 ****
  			    i2++;
  			}
  		      else
! 			i2 = i;
  
  		      if(i2 - i >1)
  			{
--- 363,369 ----
  			    i2++;
  			}
  		      else
! 			i2 = i - 1; /* grm: need to leave at space */
  
  		      if(i2 - i >1)
  			{
***************
*** 379,385 ****
  #if 0
  			  i2 = 4; /* ckh - should this be i2++ instead? */
  #else
!                           i2++;
  #endif
  			}
  		      tmp=safemalloc(strlen(Buttons[CurrentButton].action));
--- 379,385 ----
  #if 0
  			  i2 = 4; /* ckh - should this be i2++ instead? */
  #else
!                           /*i2++; grm: should not be done, see i3= i2+1 below*/
  #endif
  			}
  		      tmp=safemalloc(strlen(Buttons[CurrentButton].action));

Grant McDorman <[email-masked]>
				ISG Technologies, Inc.
--
To unsubscribe from the list, send "unsubscribe fvwm" in the body of a
message to [email-masked].
To report problems, send mail to [email-masked].