Use proper max in lwlib

[sxemacs] / src / ui / lwlib / lwlib.c

diff --git a/src/ui/lwlib/lwlib.c b/src/ui/lwlib/lwlib.c
index f0d257b..b23e76d 100644 (file)
--- a/src/ui/lwlib/lwlib.c
+++ b/src/ui/lwlib/lwlib.c
@@ -318,12 +318,15 @@ widget_value *replace_widget_value_tree(widget_value * node,
 
        if (!node || !newtree)
                abort();
+       else {
 
-       copy = copy_widget_value_tree(newtree, STRUCTURAL_CHANGE);
+               copy = copy_widget_value_tree(newtree, STRUCTURAL_CHANGE);
 
-       free_widget_value_contents(node);
-       *node = *copy;
-       free_widget_value(copy);        /* free the node, but not its contents. */
+               free_widget_value_contents(node);
+               *node = *copy;
+               /* free the node, but not its contents. */
+               free_widget_value(copy);
+       }
        return node;
 }
 
@@ -447,8 +450,7 @@ lw_map_widget_values(LWLIB_ID id, int (*mapfunc) (widget_value * value,
 
        if (!info)
                abort();
-
-       if (info->val)
+       else if (info->val)
                return map_widget_values(info->val, mapfunc, closure);
        return 0;
 }
@@ -498,6 +500,12 @@ static Boolean safe_strcmp(const char *s1, const char *s2)
        return (s1 && s2) ? strcmp(s1, s2) : s1 ? False : !!s2;
 }
 
+
+/* We only expect to use the following max function */
+#ifdef max
+#undef max
+#endif
+
 static change_type max(change_type i1, change_type i2)
 {
        return (int)i1 > (int)i2 ? i1 : i2;
@@ -896,10 +904,10 @@ static void instantiate_widget_instance(widget_instance * instance)
                fprintf(stderr, "No creation function for widget type %s\n",
                        instance->info->type);
                abort();
+       } else {
+               instance->widget = (*function) (instance);
        }
 
-       instance->widget = (*function) (instance);
-
        if (!instance->widget)
                abort();