- replaced YES/NO with true/false
[supertux.git] / src / menu.cpp
index ce2b3b8..2718246 100644 (file)
@@ -31,9 +31,9 @@
 #include "high_scores.h"
 
 /* (global) menu variables */
-int menuaction;
-int show_menu;
-int menu_change;
+MenuAction menuaction;
+bool show_menu;
+bool menu_change;
 texture_type checkbox, checkbox_checked, back, arrow_left, arrow_right;
 
 menu_type main_menu, game_menu, options_menu, highscore_menu, load_game_menu, save_game_menu;
@@ -48,7 +48,7 @@ void menu_set_current(menu_type* pmenu)
 {
   if(pmenu != current_menu)
     {
-      menu_change = YES;
+      menu_change = true;
       last_menu = current_menu;
       current_menu = pmenu;
       timer_start(&pmenu->effect, 500);
@@ -56,7 +56,7 @@ void menu_set_current(menu_type* pmenu)
 }
 
 /* Return a pointer to a new menu item */
-menu_item_type* menu_item_create(int kind, char *text, int init_toggle, void* target_menu)
+menu_item_type* menu_item_create(MenuItemKind kind, char *text, int init_toggle, void* target_menu)
 {
   menu_item_type *pnew_item = (menu_item_type*) malloc(sizeof(menu_item_type));
   pnew_item->kind = kind;
@@ -65,7 +65,7 @@ menu_item_type* menu_item_create(int kind, char *text, int init_toggle, void* ta
   if(kind == MN_TOGGLE)
     pnew_item->toggled = init_toggle;
   else
-    pnew_item->toggled = NO;
+    pnew_item->toggled = false;
   pnew_item->target_menu = target_menu;
   pnew_item->input = (char*) malloc(sizeof(char));
   pnew_item->input[0] = '\0';
@@ -117,18 +117,20 @@ void menu_free(menu_type* pmenu)
 /* Initialize a menu */
 void menu_init(menu_type* pmenu)
 {
+  pmenu->x            = screen->w/2;
+  pmenu->y            = screen->h/2;
   pmenu->arrange_left = 0;
-  pmenu->num_items = 0;
-  pmenu->active_item = 0;
-  pmenu->item = NULL;
-  timer_init(&pmenu->effect,NO);
+  pmenu->num_items    = 0;
+  pmenu->active_item  = 0;
+  pmenu->item         = NULL;
+  timer_init(&pmenu->effect,false);
 }
 
 /* Add an item to a menu */
 void menu_additem(menu_type* pmenu, menu_item_type* pmenu_item)
 {
   ++pmenu->num_items;
-  pmenu->item = (menu_item_type*) realloc(pmenu->item, sizeof(menu_item_type) * pmenu->num_items);
+  pmenu->item = (menu_item_type*)realloc(pmenu->item, sizeof(menu_item_type) * pmenu->num_items);
   memcpy(&pmenu->item[pmenu->num_items-1],pmenu_item,sizeof(menu_item_type));
   free(pmenu_item);
 }
@@ -136,119 +138,138 @@ void menu_additem(menu_type* pmenu, menu_item_type* pmenu_item)
 /* Process actions done on the menu */
 void menu_action(menu_type* pmenu)
 {
-  int i;
+  menu_item_type& item = pmenu->item[pmenu->active_item];
 
   if(pmenu->num_items != 0 && pmenu->item != NULL)
     {
       switch(menuaction)
         {
-        case MN_UP:
+        case MENU_ACTION_UP:
           if(pmenu->active_item > 0)
             --pmenu->active_item;
           else
             pmenu->active_item = pmenu->num_items-1;
           break;
-        case MN_DOWN:
+
+        case MENU_ACTION_DOWN:
           if(pmenu->active_item < pmenu->num_items-1)
             ++pmenu->active_item;
           else
             pmenu->active_item = 0;
           break;
-        case MN_LEFT:
-          if(pmenu->item[pmenu->active_item].kind == MN_STRINGSELECT && pmenu->item[pmenu->active_item].list->num_items != 0)
+
+        case MENU_ACTION_LEFT:
+          if(item.kind == MN_STRINGSELECT
+             && item.list->num_items != 0)
             {
-              if(pmenu->item[pmenu->active_item].list->active_item > 0)
-                --pmenu->item[pmenu->active_item].list->active_item;
+              if(item.list->active_item > 0)
+                --item.list->active_item;
               else
-                pmenu->item[pmenu->active_item].list->active_item = pmenu->item[pmenu->active_item].list->num_items-1;
+                item.list->active_item = item.list->num_items-1;
             }
           break;
-        case MN_RIGHT:
-          if(pmenu->item[pmenu->active_item].kind == MN_STRINGSELECT && pmenu->item[pmenu->active_item].list->num_items != 0)
+        case MENU_ACTION_RIGHT:
+          if(item.kind == MN_STRINGSELECT 
+             && item.list->num_items != 0)
             {
-              if(pmenu->item[pmenu->active_item].list->active_item < pmenu->item[pmenu->active_item].list->num_items-1)
-                ++pmenu->item[pmenu->active_item].list->active_item;
+              if(item.list->active_item < item.list->num_items-1)
+                ++item.list->active_item;
               else
-                pmenu->item[pmenu->active_item].list->active_item = 0;
+                item.list->active_item = 0;
             }
           break;
-        case MN_HIT:
-          if(pmenu->item[pmenu->active_item].kind == MN_GOTO && pmenu->item[pmenu->active_item].target_menu != NULL)
-            menu_set_current((menu_type*)pmenu->item[pmenu->active_item].target_menu);
-          else if(pmenu->item[pmenu->active_item].kind == MN_TOGGLE)
+
+        case MENU_ACTION_HIT:
+          if(item.kind == MN_GOTO
+             && item.target_menu != NULL)
+            menu_set_current((menu_type*)item.target_menu);
+
+          else if(item.kind == MN_TOGGLE)
             {
-              pmenu->item[pmenu->active_item].toggled = !pmenu->item[pmenu->active_item].toggled;
-              menu_change = YES;
+              item.toggled = !item.toggled;
+              menu_change = true;
             }
-          else if(pmenu->item[pmenu->active_item].kind == MN_ACTION || pmenu->item[pmenu->active_item].kind == MN_TEXTFIELD || pmenu->item[pmenu->active_item].kind == MN_NUMFIELD)
+          else if(item.kind == MN_ACTION || item.kind == MN_TEXTFIELD || item.kind == MN_NUMFIELD)
             {
-              pmenu->item[pmenu->active_item].toggled = YES;
+              item.toggled = true;
             }
-          else if(pmenu->item[pmenu->active_item].kind == MN_BACK)
+          else if(item.kind == MN_BACK)
             {
               if(last_menu != NULL)
                 menu_set_current(last_menu);
             }
           break;
-        case MN_REMOVE:
-          if(pmenu->item[pmenu->active_item].kind == MN_TEXTFIELD || pmenu->item[pmenu->active_item].kind == MN_NUMFIELD)
+
+        case MENU_ACTION_REMOVE:
+          if(item.kind == MN_TEXTFIELD
+             || item.kind == MN_NUMFIELD)
             {
-              if(pmenu->item[pmenu->active_item].input != NULL)
+              if(item.input != NULL)
                 {
-                  i = strlen(pmenu->item[pmenu->active_item].input);
+                  int i = strlen(item.input);
 
                   while(delete_character > 0)  /* remove charactes */
                     {
-                      pmenu->item[pmenu->active_item].input[i-1] = '\0';
+                      item.input[i-1] = '\0';
                       delete_character--;
                     }
                 }
             }
           break;
-        case MN_INPUT:
-          if(pmenu->item[pmenu->active_item].kind == MN_TEXTFIELD || (pmenu->item[pmenu->active_item].kind == MN_NUMFIELD && mn_input_char >= '0' && mn_input_char <= '9'))
+
+        case MENU_ACTION_INPUT:
+          if(item.kind == MN_TEXTFIELD
+             || (item.kind == MN_NUMFIELD && mn_input_char >= '0' && mn_input_char <= '9'))
             {
-              if(pmenu->item[pmenu->active_item].input != NULL)
+              if(item.input != NULL)
                 {
-                  i = strlen(pmenu->item[pmenu->active_item].input);
-                  pmenu->item[pmenu->active_item].input = (char*) realloc(pmenu->item[pmenu->active_item].input,sizeof(char)*(i + 2));
-                  pmenu->item[pmenu->active_item].input[i] = mn_input_char;
-                  pmenu->item[pmenu->active_item].input[i+1] = '\0';
+                  int i = strlen(item.input);
+                  item.input = (char*) realloc(item.input,sizeof(char)*(i + 2));
+                  item.input[i] = mn_input_char;
+                  item.input[i+1] = '\0';
                 }
               else
                 {
-                  pmenu->item[pmenu->active_item].input = (char*) malloc(2*sizeof(char));
-                  pmenu->item[pmenu->active_item].input[0] = mn_input_char;
-                  pmenu->item[pmenu->active_item].input[1] = '\0';
+                  item.input = (char*) malloc(2*sizeof(char));
+                  item.input[0] = mn_input_char;
+                  item.input[1] = '\0';
                 }
             }
           break;
+
+        case MENU_ACTION_NONE:
+          break;
         }
     }
 
-  if(pmenu->item[pmenu->active_item].kind == MN_DEACTIVE || pmenu->item[pmenu->active_item].kind == MN_LABEL || pmenu->item[pmenu->active_item].kind == MN_HL)
+  menu_item_type& new_item = pmenu->item[pmenu->active_item];
+  if(new_item.kind == MN_DEACTIVE
+     || new_item.kind == MN_LABEL 
+     || new_item.kind == MN_HL)
     {
-      if(menuaction != MN_UP && menuaction != MN_DOWN)
-        menuaction = MN_DOWN;
+      // Skip the horzontal line item
+      if(menuaction != MENU_ACTION_UP && menuaction != MENU_ACTION_DOWN)
+        menuaction = MENU_ACTION_DOWN;
 
       if(pmenu->num_items > 1)
         menu_action(pmenu);
     }
-
 }
 
 /* Check, if the value of the active menu item has changed. */
 int menu_check(menu_type* pmenu)
 {
+  menu_item_type& item = pmenu->item[pmenu->active_item];
+
   if(pmenu->num_items != 0 && pmenu->item != NULL)
     {
-      if((pmenu->item[pmenu->active_item].kind == MN_ACTION || pmenu->item[pmenu->active_item].kind == MN_TEXTFIELD || pmenu->item[pmenu->active_item].kind == MN_NUMFIELD) && pmenu->item[pmenu->active_item].toggled == YES)
+      if((item.kind == MN_ACTION || item.kind == MN_TEXTFIELD || item.kind == MN_NUMFIELD) && item.toggled == true)
         {
-          pmenu->item[pmenu->active_item].toggled = NO;
+          item.toggled = false;
           show_menu = 0;
           return pmenu->active_item;
         }
-      else if(pmenu->item[pmenu->active_item].kind == MN_TOGGLE || pmenu->item[pmenu->active_item].kind == MN_GOTO)
+      else if(item.kind == MN_TOGGLE || item.kind == MN_GOTO)
         {
           return pmenu->active_item;
         }
@@ -259,200 +280,203 @@ int menu_check(menu_type* pmenu)
     return -1;
 }
 
-void menu_draw_item(menu_type* pmenu, const menu_item_type& pitem,
-                    int offset, // Position of the current item in the menu
+void menu_draw_item(menu_type* pmenu, 
+                    int index, // Position of the current item in the menu
                     int menu_width, 
                     int menu_height)
 {
-  int a;
-  int b;
-  int e;
-  int f;
-
-  if(pmenu->arrange_left == YES)
-    a = menu_width * 16;
-  else
-    a = 0;
-  
-  if(pmenu->arrange_left == YES)
-    b = (a - ((strlen(pitem.text)+strlen(pitem.input)+ strlen(string_list_active(pitem.list))) * 16)) / 2;
-  else
-    b = 0;
+  int font_width  = 16;
 
-  if(timer_check(&pmenu->effect))
-    e = timer_get_left(&pmenu->effect) / 4;
-  else
-    e = 0;
+  const menu_item_type& pitem =  pmenu->item[index];
 
-  if(e != 0)
-    f = (offset % 2) ? e : -e;
-  else
-    f = 0;
+  int effect_offset = 0;
+  {
+    int effect_time = 0;
+    if(timer_check(&pmenu->effect))
+      effect_time = timer_get_left(&pmenu->effect) / 4;
 
-  int x_pos = screen->w/2 - menu_width/2;
-  int y_pos = offset*24 - menu_height/2;
+    effect_offset = (index % 2) ? effect_time : -effect_time;
+  }
 
-  int center_x = screen->w/2;
-  int center_y = screen->h /2;
-  int start_x = center_x - b;
+  int x_pos       = pmenu->x;
+  int y_pos       = pmenu->y + 24*index - menu_height/2 + 12 + effect_offset;
+  int shadow_size = 2;
+  int text_width  = strlen(pitem.text) * font_width;
+  int input_width = strlen(pitem.input) * font_width;
+  int list_width  = strlen(string_list_active(pitem.list)) * font_width;
+  text_type* text_font = &white_text;
 
-  if(pitem.kind == MN_DEACTIVE)
-    {
-      text_drawf(&black_text, pitem.text, 
-                 -b, y_pos + 10 + f,
-                 A_HMIDDLE, A_VMIDDLE, 2, NO_UPDATE);
-    }
-  else if(pitem.kind == MN_HL)
-    {
-      /* Draw a horizontal line with a little 3d effect */
-      fillrect(x_pos, y_pos + 6 + center_y,
-               menu_width, 4,
-               210,50,50,225);
-      fillrect(x_pos, y_pos + 10 + center_y, 
-               menu_width, 2,
-               0,0,0,255);
-    }
-  else if(pitem.kind == MN_LABEL)
-    {
-      text_drawf(&white_big_text, pitem.text, 
-                 -b, y_pos + 10,
-                 A_HMIDDLE, A_VMIDDLE, 2);
-    }
-  else if(pitem.kind == MN_TEXTFIELD || pitem.kind == MN_NUMFIELD)
-    {
-      int input_pos = ((strlen(pitem.input)*16)/2);
-      int text_pos  = ((strlen(pitem.text) + 1)*16)/2;
-      fillrect(-b +center_x - input_pos + text_pos - 1,y_pos + 10 + center_y - 10 + f,
-               (strlen(pitem.input)+1)*16 + 2, 20,
-               255,255,255,255);
-      fillrect(-b +center_x - input_pos + text_pos, y_pos + 10 + center_y - 9 + f,
-               (strlen(pitem.input)+1)*16, 18,
-               0,0,0,128);
-      text_drawf(&gold_text, pitem.input,
-                 -b + text_pos, y_pos + 10 + f, 
-                 A_HMIDDLE, A_VMIDDLE, 2);
-
-      if(offset == pmenu->active_item)
-        {
-          text_drawf(&blue_text,pitem.text, -b  -(((strlen(pitem.input)+1) * 16)/2),y_pos + 10 + f,A_HMIDDLE, A_VMIDDLE,3);
-        }
-      else
-        {
-          text_drawf(&white_text,pitem.text, -b  -(((strlen(pitem.input)+1) * 16)/2),y_pos +10 + f,A_HMIDDLE, A_VMIDDLE,2);
-        }
-    }
-  else if(pitem.kind == MN_STRINGSELECT)
-    {
-      int list_pos = ((strlen(string_list_active(pitem.list))*16)/2);
-      int list_pos_2 = (strlen(string_list_active(pitem.list))+1)*16;
-      int text_pos = ((strlen(pitem.text) + 1)*16)/2;
-
-      /* Draw arrows */
-      texture_draw(&arrow_left, start_x - list_pos + text_pos - 17, y_pos + 10 + center_y -8 + f);
-      texture_draw(&arrow_right, start_x - list_pos + text_pos - 1 + list_pos_2, y_pos + 10 + center_y -8 + f);
-
-      /* Draw input background */
-      fillrect(start_x - list_pos + text_pos - 1, y_pos + 10 + center_y - 10 + f,
-               list_pos_2 + 2, 20,
-               255,255,255,255);
-      fillrect(start_x - list_pos + text_pos, y_pos + 10 + center_y - 9 + f,
-               list_pos_2, 18,
-               0,0,0,128);
-
-      text_drawf(&gold_text, string_list_active(pitem.list), 
-                 -b + text_pos, y_pos + 10 + f,
-                 A_HMIDDLE, A_VMIDDLE,2);
-
-      if(offset == pmenu->active_item)
-        {
-          text_drawf(&blue_text,pitem.text,
-                     -b - list_pos_2/2, y_pos + 10 + f,
-                     A_HMIDDLE, A_VMIDDLE,3);
-        }
-      else
-        {
-          text_drawf(&white_text,pitem.text,
-                     -b - list_pos_2/2, y_pos +10 + f,
-                     A_HMIDDLE, A_VMIDDLE,2);
-        }
-    }
-  else if(offset == pmenu->active_item)
-    {
-      text_drawf(&blue_text, pitem.text, 
-                 -b,y_pos + 10 + f,
-                 A_HMIDDLE, A_VMIDDLE, 3);
-    }
-  else
+  if(pmenu->arrange_left == true)
+    x_pos += 24 - menu_width/2 + (text_width + input_width + list_width)/2;
+  
+  if(index == pmenu->active_item)
     {
-      text_drawf(&white_text, pitem.text,
-                 -b, y_pos + 10 + f, 
-                 A_HMIDDLE, A_VMIDDLE, 2);
+      shadow_size = 3;
+      text_font = &blue_text;
     }
 
-  if(pitem.kind == MN_TOGGLE)
+  switch (pitem.kind)
     {
-      if(pitem.toggled == YES)
-        texture_draw(&checkbox_checked, -b + screen->w / 2 + (strlen(pitem.text) * 16)/2  + 16,y_pos + 10 + center_y -8 + f);
-      else
-        texture_draw(&checkbox, -b + screen->w / 2 + (strlen(pitem.text) * 16)/2 + 16,y_pos + 10 + center_y - 8 + f);
+    case MN_DEACTIVE:
+      {
+        text_draw_align(&black_text, pitem.text, 
+                        x_pos, y_pos,
+                        A_HMIDDLE, A_VMIDDLE, 2);
+        break;
+      }
+
+    case MN_HL:
+      {
+        int x = pmenu->x - menu_width/2; 
+        int y = y_pos - 12 - effect_offset;
+        /* Draw a horizontal line with a little 3d effect */
+        fillrect(x, y + 6,
+                 menu_width, 4,
+                 210,50,50,225);
+        fillrect(x, y + 6, 
+                 menu_width, 2,
+                 0,0,0,255);
+        break;
+      }
+    case MN_LABEL:
+      {
+        text_draw_align(&white_big_text, pitem.text, 
+                        x_pos, y_pos,
+                        A_HMIDDLE, A_VMIDDLE, 2);
+        break;
+      }
+    case MN_TEXTFIELD:
+    case MN_NUMFIELD:
+      {
+        int input_pos = input_width/2;
+        int text_pos  = (text_width + font_width)/2;
+
+        fillrect(x_pos - input_pos + text_pos - 1, y_pos - 10,
+                 input_width + font_width + 2, 20,
+                 255,255,255,255);
+        fillrect(x_pos - input_pos + text_pos, y_pos - 9,
+                 input_width + font_width, 18,
+                 0,0,0,128);
+
+        text_draw_align(&gold_text, pitem.input,
+                        x_pos + text_pos, y_pos, 
+                        A_HMIDDLE, A_VMIDDLE, 2);
+
+        text_draw_align(text_font, pitem.text, 
+                        x_pos - (input_width + font_width)/2, y_pos,
+                        A_HMIDDLE, A_VMIDDLE, shadow_size);
+        break;
+      }
+    case MN_STRINGSELECT:
+      {
+        int list_pos_2 = list_width + font_width;
+        int list_pos   = list_width/2;
+        int text_pos   = (text_width + font_width)/2;
+
+        /* Draw arrows */
+        texture_draw(&arrow_left,  x_pos - list_pos + text_pos - 17, y_pos - 8);
+        texture_draw(&arrow_right, x_pos - list_pos + text_pos - 1 + list_pos_2, y_pos - 8);
+
+        /* Draw input background */
+        fillrect(x_pos - list_pos + text_pos - 1, y_pos - 10,
+                 list_pos_2 + 2, 20,
+                 255,255,255,255);
+        fillrect(x_pos - list_pos + text_pos, y_pos - 9,
+                 list_pos_2, 18,
+                 0,0,0,128);
+
+        text_draw_align(&gold_text, string_list_active(pitem.list), 
+                        x_pos + text_pos, y_pos,
+                        A_HMIDDLE, A_VMIDDLE,2);
+
+        text_draw_align(text_font, pitem.text,
+                        x_pos - list_pos_2/2, y_pos,
+                        A_HMIDDLE, A_VMIDDLE, shadow_size);
+        break;
+      }
+    case MN_BACK:
+      {
+        text_draw_align(text_font, pitem.text, x_pos, y_pos, A_HMIDDLE, A_VMIDDLE, shadow_size);
+        texture_draw(&back, x_pos + text_width/2  + font_width, y_pos - 8);
+        break;
+      }
+
+    case MN_TOGGLE:
+      {
+        text_draw_align(text_font, pitem.text, x_pos, y_pos, A_HMIDDLE, A_VMIDDLE, shadow_size);
+        
+        if(pitem.toggled == true)
+          texture_draw(&checkbox_checked, 
+                       x_pos + (text_width+font_width)/2,
+                       y_pos - 8);
+        else
+          texture_draw(&checkbox, 
+                       x_pos + (text_width+font_width)/2,
+                       y_pos - 8);
+        break;
+      }
+    case MN_ACTION:
+      text_draw_align(text_font, pitem.text, x_pos, y_pos, A_HMIDDLE, A_VMIDDLE, shadow_size);
+      break;
+
+    case MN_GOTO:
+      text_draw_align(text_font, pitem.text, x_pos, y_pos, A_HMIDDLE, A_VMIDDLE, shadow_size);
+      break;
     }
-  else if(pitem.kind == MN_BACK)
-    {
-      texture_draw(&back, -b + screen->w / 2 + (strlen(pitem.text) * 16)/2  + 16,y_pos + 10 + center_y -8 + f);
-    } 
 }
 
 /* Draw the current menu. */
 void menu_draw(menu_type* pmenu)
 {
-  int y;
   int menu_height;
   int menu_width;
   
-  /* The width of the menu has to be more than the width of the text with the most characters */
+  /* The width of the menu has to be more than the width of the text
+     with the most characters */
   menu_width = 0;
   for(int i = 0; i < pmenu->num_items; ++i)
     {
-      y = strlen(pmenu->item[i].text) + (pmenu->item[i].input ? strlen(pmenu->item[i].input) + 1 : 0) + strlen(string_list_active(pmenu->item[i].list));
-      if( y > menu_width )
+      int w = strlen(pmenu->item[i].text) + (pmenu->item[i].input ? strlen(pmenu->item[i].input) + 1 : 0) + strlen(string_list_active(pmenu->item[i].list));
+      if( w > menu_width )
         {
-          menu_width = y;
+          menu_width = w;
           if( pmenu->item[i].kind == MN_TOGGLE)
             menu_width += 2;
         }
     }
 
-  menu_width = menu_width * 16 + 48;
+  menu_width  = menu_width * 16 + 48;
   menu_height = (pmenu->num_items) * 24;
 
-  int center_x = screen->w/2;
   /* Draw a transparent background */
-  fillrect(center_x - menu_width/2,screen->h/2-(((pmenu->num_items)*24)/2),menu_width,menu_height,150,150,150,100);
+  fillrect(pmenu->x - menu_width/2,
+           pmenu->y - 24*pmenu->num_items/2,
+           menu_width,menu_height,150,150,150,100);
 
   for(int i = 0; i < pmenu->num_items; ++i)
     {
-      menu_draw_item(pmenu, pmenu->item[i], i, menu_width, menu_height);
+      menu_draw_item(pmenu, i, menu_width, menu_height);
     }
 }
 
 /* Reset/Set global defaults */
 void menu_reset(void)
 {
-  menu_change = NO;
-  show_menu = NO;
-  menuaction = -1;
+  menu_change  = false;
+  show_menu    = false;
+  menuaction   = MENU_ACTION_NONE;
   current_menu = NULL;
-  last_menu = NULL;
+  last_menu    = NULL;
 
   delete_character = 0;
-  mn_input_char = '\0';
+  mn_input_char    = '\0';
 }
 
 /* --- MENU --- */
 /* Draw the current menu and execute the (menu)events */
 void menu_process_current(void)
 {
-  menu_change = NO;
+  menu_change = false;
 
   if(current_menu != NULL)
     {
@@ -460,7 +484,7 @@ void menu_process_current(void)
       menu_draw(current_menu);
     }
 
-  menuaction = -1;
+  menuaction = MENU_ACTION_NONE;
 }
 
 /* Check for menu event */
@@ -486,44 +510,44 @@ void menu_event(SDL_keysym* keysym)
   switch(key)
     {
     case SDLK_UP:              /* Menu Up */
-      menuaction = MN_UP;
-      menu_change = YES;
+      menuaction = MENU_ACTION_UP;
+      menu_change = true;
       break;
     case SDLK_DOWN:            /* Menu Down */
-      menuaction = MN_DOWN;
-      menu_change = YES;
+      menuaction = MENU_ACTION_DOWN;
+      menu_change = true;
       break;
     case SDLK_LEFT:            /* Menu Up */
-      menuaction = MN_LEFT;
-      menu_change = YES;
+      menuaction = MENU_ACTION_LEFT;
+      menu_change = true;
       break;
     case SDLK_RIGHT:           /* Menu Down */
-      menuaction = MN_RIGHT;
-      menu_change = YES;
+      menuaction = MENU_ACTION_RIGHT;
+      menu_change = true;
       break;
     case SDLK_SPACE:
       if(current_menu->item[current_menu->active_item].kind == MN_TEXTFIELD)
       {
-      menuaction = MN_INPUT;
-      menu_change = YES;
+      menuaction = MENU_ACTION_INPUT;
+      menu_change = true;
       mn_input_char = ' ';
       break;
       }
     case SDLK_RETURN: /* Menu Hit */
-      menuaction = MN_HIT;
-      menu_change = YES;
+      menuaction = MENU_ACTION_HIT;
+      menu_change = true;
       break;
     case SDLK_DELETE:
     case SDLK_BACKSPACE:
-      menuaction = MN_REMOVE;
-      menu_change = YES;
+      menuaction = MENU_ACTION_REMOVE;
+      menu_change = true;
       delete_character++;
       break;
     default:
       if( (key >= SDLK_0 && key <= SDLK_9) || (key >= SDLK_a && key <= SDLK_z) || (key >= SDLK_SPACE && key <= SDLK_SLASH))
         {
-          menuaction = MN_INPUT;
-          menu_change = YES;
+          menuaction = MENU_ACTION_INPUT;
+          menu_change = true;
           mn_input_char = *ch;
         }
       else
@@ -545,3 +569,12 @@ void menu_event(SDL_keysym* keysym)
   #endif*/
 }
 
+menu_item_type*
+menu_additem(menu_type* pmenu, MenuItemKind kind, char *text, int init_toggle, void* target_menu)
+{
+  menu_item_type* item = menu_item_create(kind, text, init_toggle, target_menu);
+  menu_additem(pmenu, item);
+  return item;
+}
+
+// EOF //