ui/ncurses: Separate menu item creation & initialisation from insertion

[petitboot] / ui / ncurses / nc-menu.c

diff --git a/ui/ncurses/nc-menu.c b/ui/ncurses/nc-menu.c
index 654c389b4b030114aea346df5e0440be8a5c70ff..5691f4284942bba265ad7f87a2857ce615fd03b5 100644 (file)
--- a/ui/ncurses/nc-menu.c
+++ b/ui/ncurses/nc-menu.c
@@ -77,43 +77,39 @@ static void pmenu_resize(struct nc_scr *scr)
 }
 
 /**
- * pmenu_item_init - Allocate and initialize a new pmenu_item instance.
+ * pmenu_item_create - Allocate and initialize a new pmenu_item instance.
  *
  * Returns a pointer the the initialized struct pmenu_item instance or NULL
  * on error. The caller is responsible for calling talloc_free() for the
  * returned instance.
  */
-
-struct pmenu_item *pmenu_item_alloc(struct pmenu *menu)
+struct pmenu_item *pmenu_item_create(struct pmenu *menu, const char *name)
 {
-       /* Items go with the menu, not the pointer array. */
-
-       struct pmenu_item *i = talloc_zero(menu, struct pmenu_item);
+       struct pmenu_item *item = talloc_zero(menu, struct pmenu_item);
 
-       return i;
-}
-
-struct pmenu_item *pmenu_item_setup(struct pmenu *menu, struct pmenu_item *i,
-       unsigned int index, const char *name)
-{
-       assert(i);
-       assert(name);
+       item->i_sig = pb_item_sig;
+       item->pmenu = menu;
+       item->nci = new_item(name, NULL);
 
-       if (!i)
+       if (!item->nci) {
+               talloc_free(item);
                return NULL;
+       }
 
-       i->i_sig = pb_item_sig;
-       i->pmenu = menu;
-       i->nci = new_item(name, NULL);
-
-       if (!i->nci)
-               return NULL;
+       set_item_userptr(item->nci, item);
 
-       set_item_userptr(i->nci, i);
+       return item;
+}
 
-       menu->items[index] = i->nci;
+void pmenu_item_insert(struct pmenu *menu, struct pmenu_item *item,
+       unsigned int index)
+{
+       assert(item);
+       assert(index < menu->item_count);
+       assert(menu->items[index] == NULL);
+       assert(menu_items(menu->ncm) == NULL);
 
-       return i;
+       menu->items[index] = item->nci;
 }
 
 static int pmenu_item_get_index(const struct pmenu_item *item)
@@ -129,54 +125,6 @@ static int pmenu_item_get_index(const struct pmenu_item *item)
        return -1;
 }
 
-/**
- * pmenu_item_replace - Replace the menu item with a new one.
- *
- * Use this routine to change a menu item's text.
- */
-
-int pmenu_item_replace(struct pmenu_item *i, const char *name)
-{
-       struct pmenu *menu;
-       ITEM *nci;
-       int index;
-
-       assert(name);
-       assert(i->nci);
-
-       menu = i->pmenu;
-       index = pmenu_item_get_index(i);
-
-       if (index < 0) {
-               assert(0 && "get_index failed");
-               return -1;
-       }
-
-       nci = new_item(name, NULL);
-
-       if (!nci) {
-               assert(0 && "new_item failed");
-               return -1;
-       }
-
-       set_item_userptr(nci, i);
-
-       nc_scr_unpost(&menu->scr);
-       set_menu_items(menu->ncm, NULL);
-
-       // FIXME: need to assure item name is a talloc string.
-       /* talloc_free((char *)item_name(i->nci)); */
-
-       free_item(i->nci);
-       menu->items[index] = nci;
-       i->nci = nci;
-
-       set_menu_items(menu->ncm, menu->items);
-       nc_scr_post(&menu->scr);
-
-       return 0;
-}
-
 /**
  * pmenu_move_cursor - Move the cursor.
  * @req: An ncurses request or char to send to menu_driver().