* xmenu.c (set_frame_menubar): Use EMACS_UINT, not int, for indexes
authorPaul Eggert <eggert@cs.ucla.edu>
Tue, 22 Mar 2011 08:59:50 +0000 (01:59 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Tue, 22 Mar 2011 08:59:50 +0000 (01:59 -0700)
that could (at least in theory) be that large.

src/ChangeLog
src/xmenu.c

index fd135de..c8fa1e5 100644 (file)
@@ -1,5 +1,8 @@
 2011-03-22  Paul Eggert  <eggert@cs.ucla.edu>
 
+       * xmenu.c (set_frame_menubar): Use EMACS_UINT, not int, for indexes
+       that could (at least in theory) be that large.
+
        * xdisp.c (message_log_check_duplicate): Return unsigned long, not int.
        This is less likely to overflow, and avoids undefined behavior if
        overflow does occur.  All callers changed.  Use strtoul to scan
index eab7bb0..60ac27a 100644 (file)
@@ -922,7 +922,7 @@ set_frame_menubar (FRAME_PTR f, int first_time, int deep_p)
 #endif
   Lisp_Object items;
   widget_value *wv, *first_wv, *prev_wv = 0;
-  int i, last_i = 0;
+  EMACS_UINT i, last_i = 0;
   int *submenu_start, *submenu_end;
   int *submenu_top_level_items, *submenu_n_panes;