Subject: 2.0.44-almost: height fixit update, full patch

From: Andrew Veliath <[email-masked]>
Date: Sat, 11 Jan 1997 08:36:25 -0500 (EST)

There was a minor bug in the height.diff patch I posted a few days ago
which caused a [harmless] "Permission denied" message which shouldn't
have been there.  This patch also makes DestroyDecor use ChangeDecor
which makes the decor changing consistent when killing decors.  It's a
small but important change.

These fixes go right on top of height.diff.

My full diff to fvwm-2.0.44-almost which contains all the changes
I've posted earlier (including this one) is at
http://www.rpi.edu/~veliaa/private/andrew.diff.gz .

Regards,
Andrew

--- ../../orig/fvwm-2.0.44-almost/fvwm/builtins.c	Sat Jan 11 04:23:40 1997
+++ builtins.c	Sat Jan 11 06:24:17 1997
@@ -2421,8 +2421,9 @@
  ****************************************************************************/
 void AddToDecor(FvwmDecor *fl, char *s)
 {
-    if (!s)
-	return;
+    if (!s) return;
+    while (*s&&isspace(*s))++s;
+    if (!*s) return;
     cur_decor = fl;
     ExecuteFunction(s,NULL,&Event,C_ROOT,-1);
     cur_decor = NULL;
@@ -2499,14 +2500,13 @@
     
     if (found && (found != &Scr.DefaultDecor)) {
 	FvwmWindow *fw = Scr.FvwmRoot.next;
-	
-	prev->next = found->next;
 	while(fw != NULL)
 	{
 	    if (fw->fl == found)
-		fw->fl = &Scr.DefaultDecor;
+		ExecuteFunction("ChangeDecor Default",fw,eventp,C_WINDOW,*Module);
 	    fw = fw->next;
 	}
+	prev->next = found->next;
 	DestroyFvwmDecor(found);
 	free(found);
     }

--
Visit the official FVWM web page at <URL:http://www.hpc.uh.edu/fvwm/>.
To unsubscribe from the list, send "unsubscribe fvwm-workers" in the
body of a message to [email-masked].
To report problems, send mail to [email-masked].