emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Emacs-diffs] trunk r115277: * configure.ac (HAVE_MENUS): Remove.


From: Stefan Monnier
Subject: [Emacs-diffs] trunk r115277: * configure.ac (HAVE_MENUS): Remove.
Date: Fri, 29 Nov 2013 01:23:02 +0000
User-agent: Bazaar (2.6b2)

------------------------------------------------------------
revno: 115277
revision-id: address@hidden
parent: address@hidden
committer: Stefan Monnier <address@hidden>
branch nick: trunk
timestamp: Thu 2013-11-28 20:22:40 -0500
message:
  * configure.ac (HAVE_MENUS): Remove.
  * src/xmenu.c (Fmenu_or_popup_active_p):
  * src/window.c (Fset_window_configuration):
  * src/menu.c (Fx_popup_menu, Fx_popup_dialog):
  * src/keyboard.c (record_menu_key, read_char_x_menu_prompt):
  * src/fns.c (Fyes_or_no_p):
  * src/editfns.c (Fmessage_box, Fmessage_or_box):
  * src/alloc.c (make_save_ptr_ptr):
  * src/xdisp.c, src/w32menu.c, src/term.c, src/xterm.h, src/xterm.c:
  Remove HAVE_MENUS.
modified:
  ChangeLog                      changelog-20091113204419-o5vbwnq5f7feedwu-1538
  configure.ac                   
configure.in-20091113204419-o5vbwnq5f7feedwu-783
  nt/inc/ms-w32.h                msw32.h-20091113204419-o5vbwnq5f7feedwu-807
  src/ChangeLog                  changelog-20091113204419-o5vbwnq5f7feedwu-1438
  src/alloc.c                    alloc.c-20091113204419-o5vbwnq5f7feedwu-252
  src/editfns.c                  editfns.c-20091113204419-o5vbwnq5f7feedwu-255
  src/fns.c                      fns.c-20091113204419-o5vbwnq5f7feedwu-203
  src/keyboard.c                 keyboard.c-20091113204419-o5vbwnq5f7feedwu-449
  src/menu.c                     menu.c-20091113204419-o5vbwnq5f7feedwu-8676
  src/term.c                     term.c-20091113204419-o5vbwnq5f7feedwu-220
  src/w32menu.c                  w32menu.c-20091113204419-o5vbwnq5f7feedwu-947
  src/window.c                   window.c-20091113204419-o5vbwnq5f7feedwu-231
  src/xdisp.c                    xdisp.c-20091113204419-o5vbwnq5f7feedwu-240
  src/xmenu.c                    xmenu.c-20091113204419-o5vbwnq5f7feedwu-161
  src/xterm.c                    xterm.c-20091113204419-o5vbwnq5f7feedwu-244
  src/xterm.h                    xterm.h-20091113204419-o5vbwnq5f7feedwu-228
=== modified file 'ChangeLog'
--- a/ChangeLog 2013-11-28 23:50:09 +0000
+++ b/ChangeLog 2013-11-29 01:22:40 +0000
@@ -1,3 +1,7 @@
+2013-11-29  Stefan Monnier  <address@hidden>
+
+       * configure.ac (HAVE_MENUS): Remove.
+
 2013-11-28  Glenn Morris  <address@hidden>
 
        * configure.ac (PATH_SEP): Replace with pre-existing SEPCHAR.

=== modified file 'configure.ac'
--- a/configure.ac      2013-11-28 23:50:09 +0000
+++ b/configure.ac      2013-11-29 01:22:40 +0000
@@ -1866,9 +1866,6 @@
    fi
 fi
 
-### We always support menus.
-HAVE_MENUS=yes
-
 # Does the opsystem file prohibit the use of the GNU malloc?
 # Assume not, until told otherwise.
 GNU_MALLOC=yes
@@ -4620,8 +4617,7 @@
 fi
 AC_SUBST(LIBX_OTHER)
 
-if test "$HAVE_GTK" = yes ||
-   test "$HAVE_MENUS" != yes || test "$HAVE_X11" != yes; then
+if test "$HAVE_GTK" = yes || test "$HAVE_X11" != yes; then
   LIBXMENU=
 elif test "$USE_X_TOOLKIT" = none; then
   LIBXMENU='$(oldXMenudir)/libXMenu11.a'
@@ -4630,11 +4626,6 @@
 fi
 AC_SUBST(LIBXMENU)
 
-if test "${HAVE_MENUS}" = "yes" ; then
-  AC_DEFINE(HAVE_MENUS, 1,
-           [Define to 1 if you have mouse menus.  (This is supported in all 
configurations, but the option to specify it remains.)])
-fi
-
 if test "${GNU_MALLOC}" = "yes" ; then
   AC_DEFINE(GNU_MALLOC, 1,
            [Define to 1 if you want to use the GNU memory allocator.])

=== modified file 'nt/inc/ms-w32.h'
--- a/nt/inc/ms-w32.h   2013-10-14 16:23:10 +0000
+++ b/nt/inc/ms-w32.h   2013-11-29 01:22:40 +0000
@@ -156,9 +156,6 @@
 # ifndef HAVE_WINDOW_SYSTEM
 #  define HAVE_WINDOW_SYSTEM 1
 # endif
-# ifndef HAVE_MENUS
-#  define HAVE_MENUS 1
-# endif
 #endif
 
 /* Get some redefinitions in place.  */
@@ -176,7 +173,7 @@
 extern struct tm * sys_localtime (const time_t *);
 /* MinGW64 uses a 2-argument _setjmp, and setjmp is a macro defined to
    supply the 2nd arg correctly, so don't use _setjmp directly in that
-   case. */
+   case.  */
 #undef HAVE__SETJMP
 #endif
 

=== modified file 'src/ChangeLog'
--- a/src/ChangeLog     2013-11-29 01:14:01 +0000
+++ b/src/ChangeLog     2013-11-29 01:22:40 +0000
@@ -1,5 +1,14 @@
 2013-11-29  Stefan Monnier  <address@hidden>
 
+       * xmenu.c (Fmenu_or_popup_active_p):
+       * window.c (Fset_window_configuration):
+       * menu.c (Fx_popup_menu, Fx_popup_dialog):
+       * keyboard.c (record_menu_key, read_char_x_menu_prompt):
+       * fns.c (Fyes_or_no_p):
+       * editfns.c (Fmessage_box, Fmessage_or_box):
+       * alloc.c (make_save_ptr_ptr):
+       * xdisp.c, w32menu.c, term.c, xterm.h, xterm.c: Remove HAVE_MENUS.
+
        * window.c (Fset_window_configuration): Move select_window later.
 
 2013-11-28  Stefan Monnier  <address@hidden>

=== modified file 'src/alloc.c'
--- a/src/alloc.c       2013-11-28 22:43:09 +0000
+++ b/src/alloc.c       2013-11-29 01:22:40 +0000
@@ -3477,7 +3477,7 @@
   return val;
 }
 
-#if defined HAVE_MENUS && ! (defined USE_X_TOOLKIT || defined USE_GTK)
+#if ! (defined USE_X_TOOLKIT || defined USE_GTK)
 Lisp_Object
 make_save_ptr_ptr (void *a, void *b)
 {

=== modified file 'src/editfns.c'
--- a/src/editfns.c     2013-11-24 18:28:33 +0000
+++ b/src/editfns.c     2013-11-29 01:22:40 +0000
@@ -3473,7 +3473,6 @@
   else
     {
       Lisp_Object val = Fformat (nargs, args);
-#ifdef HAVE_MENUS
       Lisp_Object pane, menu;
       struct gcpro gcpro1;
 
@@ -3482,9 +3481,6 @@
       menu = Fcons (val, pane);
       Fx_popup_dialog (Qt, menu, Qt);
       UNGCPRO;
-#else /* !HAVE_MENUS */
-      message3 (val);
-#endif
       return val;
     }
 }
@@ -3503,11 +3499,9 @@
 usage: (message-or-box FORMAT-STRING &rest ARGS)  */)
   (ptrdiff_t nargs, Lisp_Object *args)
 {
-#ifdef HAVE_MENUS
   if ((NILP (last_nonmenu_event) || CONSP (last_nonmenu_event))
       && use_dialog_box)
     return Fmessage_box (nargs, args);
-#endif
   return Fmessage (nargs, args);
 }
 

=== modified file 'src/fns.c'
--- a/src/fns.c 2013-11-14 02:39:28 +0000
+++ b/src/fns.c 2013-11-29 01:22:40 +0000
@@ -35,11 +35,9 @@
 #include "frame.h"
 #include "window.h"
 #include "blockinput.h"
-#ifdef HAVE_MENUS
 #if defined (HAVE_X_WINDOWS)
 #include "xterm.h"
 #endif
-#endif /* HAVE_MENUS */
 
 Lisp_Object Qstring_lessp;
 static Lisp_Object Qprovide, Qrequire;
@@ -2416,7 +2414,6 @@
 
   CHECK_STRING (prompt);
 
-#ifdef HAVE_MENUS
   if ((NILP (last_nonmenu_event) || CONSP (last_nonmenu_event))
       && use_dialog_box)
     {
@@ -2430,7 +2427,6 @@
       UNGCPRO;
       return obj;
     }
-#endif /* HAVE_MENUS */
 
   args[0] = prompt;
   args[1] = build_string ("(yes or no) ");

=== modified file 'src/keyboard.c'
--- a/src/keyboard.c    2013-11-24 18:28:33 +0000
+++ b/src/keyboard.c    2013-11-29 01:22:40 +0000
@@ -3222,8 +3222,6 @@
   RETURN_UNGCPRO (c);
 }
 
-#ifdef HAVE_MENUS
-
 /* Record a key that came from a mouse menu.
    Record it for echoing, for this-command-keys, and so on.  */
 
@@ -3254,13 +3252,11 @@
   /* Record this character as part of the current key.  */
   add_command_key (c);
 
-  /* Re-reading in the middle of a command */
+  /* Re-reading in the middle of a command.  */
   last_input_event = c;
   num_input_events++;
 }
 
-#endif /* HAVE_MENUS */
-
 /* Return true if should recognize C as "the help character".  */
 
 static bool
@@ -8359,7 +8355,6 @@
   if (! menu_prompting)
     return Qnil;
 
-#ifdef HAVE_MENUS
   /* If we got to this point via a mouse click,
      use a real menu for mouse selection.  */
   if (EVENT_HAS_PARAMETERS (prev_event)
@@ -8405,7 +8400,6 @@
        *used_mouse_menu = 1;
       return value;
     }
-#endif /* HAVE_MENUS */
   return Qnil ;
 }
 

=== modified file 'src/menu.c'
--- a/src/menu.c        2013-10-16 15:07:36 +0000
+++ b/src/menu.c        2013-11-29 01:22:40 +0000
@@ -1178,7 +1178,6 @@
        keybinding equivalents, but we don't do that any more anyway.  */
     return Qnil;
 
-#ifdef HAVE_MENUS
   {
     bool get_current_pos_p = 0;
 
@@ -1315,7 +1314,6 @@
 
     XSETFRAME (Vmenu_updating_frame, f);
   }
-#endif /* HAVE_MENUS */
 
   /* Now parse the lisp menus.  */
   record_unwind_protect_void (unuse_menu_items);
@@ -1398,7 +1396,6 @@
 
   unbind_to (specpdl_count, Qnil);
 
-#ifdef HAVE_MENUS
 #ifdef HAVE_WINDOW_SYSTEM
   /* Hide a previous tip, if any.  */
   if (!FRAME_TERMCAP_P (f))
@@ -1460,16 +1457,12 @@
     FRAME_DISPLAY_INFO (f)->grabbed = 0;
 #endif
 
-#endif /* HAVE_MENUS */
-
   UNGCPRO;
 
   if (error_name) error ("%s", error_name);
   return selection;
 }
 
-#ifdef HAVE_MENUS
-
 DEFUN ("x-popup-dialog", Fx_popup_dialog, Sx_popup_dialog, 2, 3, 0,
        doc: /* Pop up a dialog box and return user's selection.
 POSITION specifies which frame to use.
@@ -1605,8 +1598,6 @@
   }
 }
 
-#endif /* HAVE_MENUS */
-
 void
 syms_of_menu (void)
 {

=== modified file 'src/term.c'
--- a/src/term.c        2013-11-06 00:14:56 +0000
+++ b/src/term.c        2013-11-29 01:22:40 +0000
@@ -2739,7 +2739,7 @@
                               Menus
  ***********************************************************************/
 
-#if defined (HAVE_MENUS) && !defined (MSDOS)
+#if !defined (MSDOS)
 
 /* TTY menu implementation and main ideas are borrowed from msdos.c.
 
@@ -3816,7 +3816,7 @@
   return entry;
 }
 
-#endif /* HAVE_MENUS && !MSDOS */
+#endif /* !MSDOS */
 
 
 #ifndef MSDOS
@@ -3825,7 +3825,7 @@
  ***********************************************************************/
 
 /* Initialize the tty-dependent part of frame F.  The frame must
-   already have its device initialized. */
+   already have its device initialized.  */
 
 void
 create_tty_output (struct frame *f)
@@ -3839,7 +3839,7 @@
   f->output_data.tty = t;
 }
 
-/* Delete frame F's face cache, and its tty-dependent part. */
+/* Delete frame F's face cache, and its tty-dependent part.  */
 
 static void
 tty_free_frame_resources (struct frame *f)
@@ -3854,7 +3854,7 @@
 
 #else  /* MSDOS */
 
-/* Delete frame F's face cache. */
+/* Delete frame F's face cache.  */
 
 static void
 tty_free_frame_resources (struct frame *f)

=== modified file 'src/w32menu.c'
--- a/src/w32menu.c     2013-11-04 06:09:03 +0000
+++ b/src/w32menu.c     2013-11-29 01:22:40 +0000
@@ -114,7 +114,6 @@
 
 void w32_free_menu_strings (HWND);
 
-#ifdef HAVE_MENUS
 #ifdef HAVE_DIALOGS
 Lisp_Object
 w32_popup_dialog (struct frame *f, Lisp_Object header, Lisp_Object contents)
@@ -1602,21 +1601,15 @@
   current_popup_menu = NULL;
 }
 
-#endif /* HAVE_MENUS */
-
 /* The following is used by delayed window autoselection.  */
 
 DEFUN ("menu-or-popup-active-p", Fmenu_or_popup_active_p, 
Smenu_or_popup_active_p, 0, 0, 0,
        doc: /* Return t if a menu or popup dialog is active on selected frame. 
 */)
   (void)
 {
-#ifdef HAVE_MENUS
   struct frame *f;
   f = SELECTED_FRAME ();
   return (f->output_data.w32->menubar_active > 0) ? Qt : Qnil;
-#else
-  return Qnil;
-#endif /* HAVE_MENUS */
 }
 
 void

=== modified file 'src/window.c'
--- a/src/window.c      2013-11-29 01:14:01 +0000
+++ b/src/window.c      2013-11-29 01:22:40 +0000
@@ -5554,7 +5554,6 @@
          || data->frame_cols != previous_frame_cols)
        change_frame_size (f, data->frame_lines,
                           data->frame_cols, 0, 0, 0);
-#ifdef HAVE_MENUS
       if (data->frame_menu_bar_lines
          != previous_frame_menu_bar_lines)
        {
@@ -5567,7 +5566,6 @@
            set_menu_bar_lines (f, make_number (data->frame_menu_bar_lines),
                                make_number (0));
        }
-#endif
 #ifdef HAVE_WINDOW_SYSTEM
       if (data->frame_tool_bar_lines
          != previous_frame_tool_bar_lines)
@@ -5752,7 +5750,6 @@
          || previous_frame_cols != FRAME_COLS (f))
        change_frame_size (f, previous_frame_lines, previous_frame_cols,
                           0, 0, 0);
-#ifdef HAVE_MENUS
       if (previous_frame_menu_bar_lines != FRAME_MENU_BAR_LINES (f))
        {
 #ifdef HAVE_WINDOW_SYSTEM
@@ -5765,7 +5762,6 @@
            set_menu_bar_lines (f, make_number (previous_frame_menu_bar_lines),
                                make_number (0));
        }
-#endif
 #ifdef HAVE_WINDOW_SYSTEM
       if (previous_frame_tool_bar_lines != FRAME_TOOL_BAR_LINES (f))
        x_set_tool_bar_lines (f, make_number (previous_frame_tool_bar_lines),
@@ -5799,7 +5795,7 @@
       /* This `select_window' calls record_buffer which calls Fdelq which
         invokes QUIT, so we do it here at the end rather than earlier,
         to minimize the risk of interrupting the Fset_window_configuration
-        in an inconsistent state (e.g. before frame-focus redirection is
+        in an inconsistent state (e.g. before frame-focus redirection is 
         canceled).  */
       select_window (data->current_window, Qnil, 1);
       BVAR (XBUFFER (XWINDOW (selected_window)->contents),

=== modified file 'src/xdisp.c'
--- a/src/xdisp.c       2013-11-28 22:43:09 +0000
+++ b/src/xdisp.c       2013-11-29 01:22:40 +0000
@@ -20603,7 +20603,6 @@
   compute_line_metrics (&it);
 }
 
-#ifdef HAVE_MENUS
 /* Deep copy of a glyph row, including the glyphs.  */
 static void
 deep_copy_glyph_row (struct glyph_row *to, struct glyph_row *from)
@@ -20725,7 +20724,6 @@
   row->full_width_p = saved_width;
   row->reversed_p = saved_reversed;
 }
-#endif /* HAVE_MENUS */
 
 /***********************************************************************
                              Mode Line

=== modified file 'src/xmenu.c'
--- a/src/xmenu.c       2013-11-04 06:09:03 +0000
+++ b/src/xmenu.c       2013-11-29 01:22:40 +0000
@@ -2440,11 +2440,7 @@
        doc: /* Return t if a menu or popup dialog is active.  */)
   (void)
 {
-#ifdef HAVE_MENUS
   return (popup_activated ()) ? Qt : Qnil;
-#else
-  return Qnil;
-#endif /* HAVE_MENUS */
 }
 
 void

=== modified file 'src/xterm.c'
--- a/src/xterm.c       2013-11-28 22:43:09 +0000
+++ b/src/xterm.c       2013-11-29 01:22:40 +0000
@@ -3528,7 +3528,7 @@
 }
 
 
-#if defined HAVE_MENUS && !defined USE_X_TOOLKIT && !defined USE_GTK
+#if !defined USE_X_TOOLKIT && !defined USE_GTK
 /* Handle an event saying the mouse has moved out of an Emacs frame.  */
 
 void

=== modified file 'src/xterm.h'
--- a/src/xterm.h       2013-10-29 05:55:25 +0000
+++ b/src/xterm.h       2013-11-29 01:22:40 +0000
@@ -949,7 +949,7 @@
 extern bool x_alloc_nearest_color (struct frame *, Colormap, XColor *);
 extern void x_query_color (struct frame *f, XColor *);
 extern void x_clear_area (Display *, Window, int, int, int, int);
-#if defined HAVE_MENUS && !defined USE_X_TOOLKIT && !defined USE_GTK
+#if !defined USE_X_TOOLKIT && !defined USE_GTK
 extern void x_mouse_leave (struct x_display_info *);
 #endif
 
@@ -963,7 +963,7 @@
 extern void x_set_sticky (struct frame *, Lisp_Object, Lisp_Object);
 extern void x_wait_for_event (struct frame *, int);
 
-/* Defined in xselect.c */
+/* Defined in xselect.c.  */
 
 extern void x_handle_property_notify (const XPropertyEvent *);
 extern void x_handle_selection_notify (const XSelectionEvent *);


reply via email to

[Prev in Thread] Current Thread [Next in Thread]