changeset 91773:5560603238a5

(push_submenu_start, push_submenu_end, push_left_right_boundary, push_menu_pane, push_menu_item): Don't pass args with side effects to AREF, it fails when compiling with -DENABLE_CHECKING.
author Juanma Barranquero <lekktu@gmail.com>
date Mon, 11 Feb 2008 11:28:35 +0000
parents db64fb3d9118
children 47c48e8e3205
files src/w32menu.c
diffstat 1 files changed, 17 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/src/w32menu.c	Mon Feb 11 10:10:46 2008 +0000
+++ b/src/w32menu.c	Mon Feb 11 11:28:35 2008 +0000
@@ -332,7 +332,8 @@
   if (menu_items_used + 1 > menu_items_allocated)
     grow_menu_items ();
 
-  ASET (menu_items, menu_items_used++, Qnil);
+  ASET (menu_items, menu_items_used, Qnil);
+  menu_items_used++;
   menu_items_submenu_depth++;
 }
 
@@ -344,7 +345,8 @@
   if (menu_items_used + 1 > menu_items_allocated)
     grow_menu_items ();
 
-  ASET (menu_items, menu_items_used++, Qlambda);
+  ASET (menu_items, menu_items_used, Qlambda);
+  menu_items_used++;
   menu_items_submenu_depth--;
 }
 
@@ -356,7 +358,8 @@
   if (menu_items_used + 1 > menu_items_allocated)
     grow_menu_items ();
 
-  ASET (menu_items, menu_items_used++, Qquote);
+  ASET (menu_items, menu_items_used, Qquote);
+  menu_items_used++;
 }
 
 /* Start a new menu pane in menu_items.
@@ -371,9 +374,9 @@
 
   if (menu_items_submenu_depth == 0)
     menu_items_n_panes++;
-  ASET (menu_items, menu_items_used++, Qt);
-  ASET (menu_items, menu_items_used++, name);
-  ASET (menu_items, menu_items_used++, prefix_vec);
+  ASET (menu_items, menu_items_used, Qt);         menu_items_used++;
+  ASET (menu_items, menu_items_used, name);       menu_items_used++;
+  ASET (menu_items, menu_items_used, prefix_vec); menu_items_used++;
 }
 
 /* Push one menu item into the current pane.  NAME is the string to
@@ -391,14 +394,14 @@
   if (menu_items_used + MENU_ITEMS_ITEM_LENGTH > menu_items_allocated)
     grow_menu_items ();
 
-  ASET (menu_items, menu_items_used++, name);
-  ASET (menu_items, menu_items_used++, enable);
-  ASET (menu_items, menu_items_used++, key);
-  ASET (menu_items, menu_items_used++, equiv);
-  ASET (menu_items, menu_items_used++, def);
-  ASET (menu_items, menu_items_used++, type);
-  ASET (menu_items, menu_items_used++, selected);
-  ASET (menu_items, menu_items_used++, help);
+  ASET (menu_items, menu_items_used, name);     menu_items_used++;
+  ASET (menu_items, menu_items_used, enable);   menu_items_used++;
+  ASET (menu_items, menu_items_used, key);      menu_items_used++;
+  ASET (menu_items, menu_items_used, equiv);    menu_items_used++;
+  ASET (menu_items, menu_items_used, def);      menu_items_used++;
+  ASET (menu_items, menu_items_used, type);     menu_items_used++;
+  ASET (menu_items, menu_items_used, selected); menu_items_used++;
+  ASET (menu_items, menu_items_used, help);     menu_items_used++;
 }
 
 /* Look through KEYMAPS, a vector of keymaps that is NMAPS long,