From: Jan Djärv <jan.h.d@swipnet.se>
Date: Sun, 26 Jan 2003 13:06:48 +0000 (+0000)
Subject: lwlib-Xm.c: Fixed memory leak for menus.
X-Git-Tag: ttn-vms-21-2-B4~11467
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=d0bdb0602c9c73e1aa2d9d6de7680352c47eeb38;p=emacs.git

lwlib-Xm.c: Fixed memory leak for menus.
---

diff --git a/lwlib/ChangeLog b/lwlib/ChangeLog
index 6649f0960d6..5f3ce50f869 100644
--- a/lwlib/ChangeLog
+++ b/lwlib/ChangeLog
@@ -1,3 +1,8 @@
+2003-01-26  Jan D.  <jan.h.d@swipnet.se>
+
+	* lwlib-Xm.c (update_one_menu_entry): Deallocate widget_list.
+	(destroy_all_children): Call it self to destroy sub menu children.
+
 2002-12-22  Richard M. Stallman  <rms@gnu.org>
 
 	* xlwmenu.c (pop_new_stack_if_no_contents): Do nothing if
diff --git a/lwlib/lwlib-Xm.c b/lwlib/lwlib-Xm.c
index c55aa473247..f61b72534df 100644
--- a/lwlib/lwlib-Xm.c
+++ b/lwlib/lwlib-Xm.c
@@ -268,7 +268,10 @@ destroy_all_children (widget, first_child_to_destroy)
 	  XtSetArg (al[0], XmNsubMenuId, &submenu); 
   	  XtGetValues (children[i], al, 1);
 	  if (submenu)
-	    XtDestroyWidget (submenu);
+            {
+              destroy_all_children (submenu, 0);
+              XtDestroyWidget (submenu);
+            }
 	  XtDestroyWidget (children[i]);
 	}
 
@@ -734,6 +737,9 @@ update_one_menu_entry (instance, widget, val, deep_p)
 			     (XtPointer)instance);
 	      XtManageChild (button);
 	    }
+
+          if (widget_list)
+            XtFree ((char*) widget_list);
 	}
     }
   else if (!contents)
@@ -813,7 +819,7 @@ xm_update_menu (instance, widget, val, deep_p)
     {
       destroy_all_children (widget, num_children_to_keep);
       make_menu_in_widget (instance, widget, val->contents,
-			   num_children_to_keep);
+                           num_children_to_keep);
     }
 
   XtFree ((char *) children);