destroy tree : initial NULL test

[lttv.git] / ltt / branches / poly / lttv / modules / gui / lttvwindow / lttvwindow / menu.c

diff --git a/ltt/branches/poly/lttv/modules/gui/lttvwindow/lttvwindow/menu.c b/ltt/branches/poly/lttv/modules/gui/lttvwindow/lttvwindow/menu.c
index 33f170b0054dc2c49a6f52fe1ee27963b2470ecd..d201ac0ec46c323748b36756a76d7d8096709288 100644 (file)
--- a/ltt/branches/poly/lttv/modules/gui/lttvwindow/lttvwindow/menu.c
+++ b/ltt/branches/poly/lttv/modules/gui/lttvwindow/lttvwindow/menu.c
@@ -20,30 +20,29 @@
 #include <lttvwindow/menu.h>
 
 
-inline LttvMenus *lttv_menus_new() {
+LttvMenus *lttv_menus_new() {
   return g_array_new(FALSE, FALSE, sizeof(LttvMenuClosure));
 }
 
 /* MD: delete elements of the array also, but don't free pointed addresses
  * (functions).
  */
-inline void lttv_menus_destroy(LttvMenus *h) {
+void lttv_menus_destroy(LttvMenus *h) {
   g_debug("lttv_menus_destroy()");
   g_array_free(h, TRUE);
 }
 
-inline LttvMenuClosure lttv_menus_add(LttvMenus *h, lttvwindow_viewer_constructor f, char* menu_path, char* menu_text, GtkWidget *widget)
+LttvMenuClosure lttv_menus_add(LttvMenus *h,
+    lttvwindow_viewer_constructor f,
+    char* menu_path, char* menu_text, GtkWidget *widget)
 {
   LttvMenuClosure c;
 
-  /* if h is null, do nothing, or popup a warning message */
-  if(h == NULL)return;
-
   c.con = f;
   c.menu_path = menu_path;
   c.menu_text = menu_text;
   c.widget = widget;
-  g_array_append_val(h,c);
+  if(h != NULL) g_array_append_val(h,c);
 
   return c;
 }
@@ -51,7 +50,7 @@ inline LttvMenuClosure lttv_menus_add(LttvMenus *h, lttvwindow_viewer_constructo
 GtkWidget *lttv_menus_remove(LttvMenus *h, lttvwindow_viewer_constructor f)
 {
   LttvMenuClosure * tmp;
-  gint i;
+  guint i;
   GtkWidget *widget;
   
   for(i=0;i<h->len;i++){