changeset 6009:f68004daa523

*** empty log message ***

--- a/lwlib/Makefile.in	Sat Feb 19 06:29:30 1994 +0000
+++ b/lwlib/Makefile.in	Sat Feb 19 20:53:26 1994 +0000
@@ -54,10 +54,10 @@
 #That's ok.  So don't stop the build.
 
 lwlib.o:  lwlib.c
-	$(CC) -c $(ALL_CFLAGS)  $(TOOLKIT_DEFINES) $(srcdir)/lwlib.c
+	$(CC) -c $(TOOLKIT_DEFINES) $(ALL_CFLAGS) $(srcdir)/lwlib.c
 
 xrdb-cpp.o:  xrdb-cpp.c
-	$(CC) -c $(ALL_CFLAGS) "-DCPP_PROGRAM=$(CPP)" $(srcdir)/xrdb-cpp.c
+	$(CC) -c "-DCPP_PROGRAM=$(CPP)" $(ALL_CFLAGS) $(srcdir)/xrdb-cpp.c
 
 lwlib-utils.o: lwlib-utils.h
 lwlib.o:       lwlib.h lwlib-int.h

--- a/lwlib/lwlib-Xlw.c	Sat Feb 19 06:29:30 1994 +0000
+++ b/lwlib/lwlib-Xlw.c	Sat Feb 19 20:53:26 1994 +0000
@@ -78,19 +78,24 @@
 {
   Widget widget;
 
-  widget_value *tem = malloc_widget_value (); 
+  /* Don't use malloc_widget_value, because the freeing will be done by free.
+     (Also it wastes time calling memset).  */
+  widget_value *tem = (widget_value *) malloc (sizeof (widget_value));
 
   /* _XtCreate is freeing the object we passed,
      so make a copy that we free later.  */
   bcopy (instance->info->val, tem, sizeof (widget_value));
 
-  widget =
-    XtVaCreateWidget (instance->info->name, xlwMenuWidgetClass,
-		      instance->parent,
-		      XtNmenu, tem,
-		      0);
+  widget
+    = XtVaCreateWidget (instance->info->name, xlwMenuWidgetClass,
+			instance->parent,
+			XtNmenu, tem,
+			0);
 
+#if 0 /* XtVaCreateWidget frees this, at least in the X11R4
+	 version that is running on mole.gnu.ai.mit.edu.  */
   free_widget_value (tem);
+#endif
 
   XtAddCallback (widget, XtNopen, pre_hook, (XtPointer)instance);
   XtAddCallback (widget, XtNselect, pick_hook, (XtPointer)instance);
@@ -101,26 +106,30 @@
 xlw_create_popup_menu (instance)
      widget_instance* instance;
 {
-  Widget popup_shell =
-    XtCreatePopupShell (instance->info->name, overrideShellWidgetClass,
-			instance->parent, NULL, 0);
+  Widget popup_shell
+    = XtCreatePopupShell (instance->info->name, overrideShellWidgetClass,
+			  instance->parent, NULL, 0);
   
   Widget widget;
 
-  widget_value *tem = malloc_widget_value ();
+  /* Don't use malloc_widget_value, because the freeing will be done by free.
+     (Also it wastes time calling memset).  */
+  widget_value *tem = (widget_value *) malloc (sizeof (widget_value));
 
   /* _XtCreate is freeing the object we passed,
      so make a copy that we free later.  */
   bcopy (instance->info->val, tem, sizeof (widget_value));
 
-  widget = 
-    XtVaCreateManagedWidget ("popup", xlwMenuWidgetClass,
-			     popup_shell,
-			     XtNmenu, tem,
-			     XtNhorizontal, False,
-			     0);
+  widget
+    = XtVaCreateManagedWidget ("popup", xlwMenuWidgetClass,
+			       popup_shell,
+			       XtNmenu, tem,
+			       XtNhorizontal, False,
+			       0);
 
+#if 0
   free_widget_value (tem);
+#endif
 
   XtAddCallback (widget, XtNselect, pick_hook, (XtPointer)instance);
 
@@ -140,12 +149,12 @@
      Widget widget;
 {
   WidgetClass the_class = XtClass (widget);
+
   if (the_class == xlwMenuWidgetClass)
     return True;
   if (the_class == overrideShellWidgetClass)
-    return
-      XtClass (((CompositeWidget)widget)->composite.children [0])
-	== xlwMenuWidgetClass;
+    return (XtClass (((CompositeWidget)widget)->composite.children [0])
+	    == xlwMenuWidgetClass);
   return False;
 }