(shell-command-on-region): Doc fix.
[bpt/emacs.git] / src / msdos.c
index 31c3cc1..f70c196 100644 (file)
@@ -214,6 +214,25 @@ mouse_released (b, xp, yp)
   return (regs.x.bx != 0);
 }
 
+static int
+mouse_button_depressed (b, xp, yp)
+     int b, *xp, *yp;
+{
+  union REGS regs;
+
+  if (b >= mouse_button_count)
+    return 0;
+  regs.x.ax = 0x0003;
+  int86 (0x33, &regs, &regs);
+  if ((regs.x.bx & (1 << mouse_button_translate[b])) != 0)
+    {
+      *xp = regs.x.cx / 8;
+      *yp = regs.x.dx / 8;
+      return 1;
+    }
+  return 0;
+}
+
 void
 mouse_get_pos (f, insist, bar_window, part, x, y, time)
      FRAME_PTR *f;
@@ -252,6 +271,7 @@ void
 mouse_init ()
 {
   union REGS regs;
+  int b;
 
   if (termscript)
     fprintf (termscript, "<M_INIT>");
@@ -259,6 +279,18 @@ mouse_init ()
   regs.x.ax = 0x0021;
   int86 (0x33, &regs, &regs);
 
+  /* Reset the mouse last press/release info.  It seems that Windows
+     doesn't do that automatically when function 21h is called, which
+     causes Emacs to ``remember'' the click that switched focus to the
+     window just before Emacs was started from that window.  */
+  for (b = 0; b < mouse_button_count; b++)
+    {
+      int dummy_x, dummy_y;
+
+      (void) mouse_pressed (b, &dummy_x, &dummy_y);
+      (void) mouse_released (b, &dummy_x, &dummy_y);
+    }
+
   regs.x.ax = 0x0007;
   regs.x.cx = 0;
   regs.x.dx = 8 * (ScreenCols () - 1);
@@ -771,15 +803,38 @@ IT_cmgoto (FRAME_PTR f)
   /* Only set the cursor to where it should be if the display is
      already in sync with the window contents.  */
   int update_cursor_pos = MODIFF == unchanged_modified;
-
-  /* If we are in the echo area, and the cursor is beyond the end of
-     the text, put the cursor at the end of text.  */
+  static int previous_pos_X = -1;
+
+  /* If the display is in sync, forget any previous knowledge about
+     cursor position.  This is primarily for unexpected events like
+     C-g in the minibuffer.  */
+  if (update_cursor_pos && previous_pos_X >= 0)
+    previous_pos_X = -1;
+  /* If we are in the echo area, put the cursor at the
+     end of the echo area message.  */
   if (!update_cursor_pos
       && XFASTINT (XWINDOW (FRAME_MINIBUF_WINDOW (f))->top) <= new_pos_Y)
     {
-      int tem_X = FRAME_DESIRED_GLYPHS (f)->used[new_pos_Y];
+      int tem_X = current_pos_X, dummy;
+
+      if (echo_area_glyphs)
+       {
+         tem_X = echo_area_glyphs_length;
+         /* Save current cursor position, to be restored after the
+            echo area message is erased.  Only remember one level
+            of previous cursor position.  */
+         if (previous_pos_X == -1)
+           ScreenGetCursor (&dummy, &previous_pos_X);
+       }
+      else if (previous_pos_X >= 0)
+       {
+         /* We wind up here after the echo area message is erased.
+            Restore the cursor position we remembered above.  */
+         tem_X = previous_pos_X;
+         previous_pos_X = -1;
+       }
 
-      if (current_pos_X > tem_X)
+      if (current_pos_X != tem_X)
        {
          new_pos_X = tem_X;
          update_cursor_pos = 1;
@@ -1284,6 +1339,7 @@ dos_set_keyboard (code, always)
      Note: calling Int 2Fh via int86 wedges the DOS box on some versions
      of Windows 9X!  So don't do that!  */
   regs.x.ax = 0xad80;
+  regs.x.ss = regs.x.sp = regs.x.flags = 0;
   _go32_dpmi_simulate_int (0x2f, &regs);
   if (regs.h.al == 0xff)
     international_keyboard = 1;
@@ -1667,6 +1723,7 @@ and then the scan code.")
 /* Get a char from keyboard.  Function keys are put into the event queue.  */
 
 extern void kbd_buffer_store_event (struct input_event *);
+static int mouse_preempted = 0;        /* non-zero when XMenu gobbles mouse events */
 
 static int
 dos_rawgetc ()
@@ -1874,7 +1931,7 @@ dos_rawgetc ()
       kbd_buffer_store_event (&event);
     }
 
-  if (have_mouse > 0)
+  if (have_mouse > 0 && !mouse_preempted)
     {
       int but, press, x, y, ok;
 
@@ -2240,6 +2297,10 @@ XMenuActivate (Display *foo, XMenu *menu, int *pane, int *selidx,
   if (y0 <= 0)
     y0 = 1;
 
+  /* We will process all the mouse events directly, so we had
+     better prevented dos_rawgetc from stealing them from us.  */
+  mouse_preempted++;
+
   state = alloca (menu->panecount * sizeof (struct IT_menu_state));
   screensize = screen_size * 2;
   faces[0]
@@ -2362,11 +2423,22 @@ XMenuActivate (Display *foo, XMenu *menu, int *pane, int *selidx,
                           state[statecount - 1].x,
                           faces);
        }
-      for (b = 0; b < mouse_button_count; b++)
+      else
+       /* We are busy-waiting for the mouse to move, so let's be nice
+          to other Windows applications by releasing our time slice.  */
+       __dpmi_yield ();
+      for (b = 0; b < mouse_button_count && !leave; b++)
        {
-         (void) mouse_pressed (b, &x, &y);
-         if (mouse_released (b, &x, &y))
-           leave = 1;
+         /* Only leave if user both pressed and released the mouse, and in
+            that order.  This avoids popping down the menu pane unless
+            the user is really done with it.  */
+         if (mouse_pressed (b, &x, &y))
+           {
+             while (mouse_button_depressed (b, &x, &y))
+               __dpmi_yield ();
+             leave = 1;
+           }
+         (void) mouse_released (b, &x, &y);
        }
     }
 
@@ -2377,6 +2449,14 @@ XMenuActivate (Display *foo, XMenu *menu, int *pane, int *selidx,
   while (statecount--)
     xfree (state[statecount].screen_behind);
   IT_display_cursor (1);       /* turn cursor back on */
+  /* Clean up any mouse events that are waiting inside Emacs event queue.
+     These events are likely to be generated before the menu was even
+     displayed, probably because the user pressed and released the button
+     (which invoked the menu) too quickly.  If we don't remove these events,
+     Emacs will process them after we return and surprise the user.  */
+  discard_mouse_events ();
+  /* Allow mouse events generation by dos_rawgetc.  */
+  mouse_preempted--;
   return result;
 }