(compute_glyph_face_1): New function.
[bpt/emacs.git] / src / xterm.c
index 86b66f8..09c05db 100644 (file)
@@ -1,5 +1,5 @@
 /* X Communication module for terminals which understand the X protocol.
-   Copyright (C) 1989, 1993 Free Software Foundation, Inc.
+   Copyright (C) 1989, 1993, 1994 Free Software Foundation, Inc.
 
 This file is part of GNU Emacs.
 
@@ -17,23 +17,20 @@ You should have received a copy of the GNU General Public License
 along with GNU Emacs; see the file COPYING.  If not, write to
 the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
-/* Serious problems:
-
-   Kludge: dup2 is used to put the X-connection socket into desc # 0
-   so that wait_reading_process_input will wait for it in place of
-   actual terminal input.
-   
-*/
-
-#define NEW_SELECTIONS
+/* Xt features made by Fred Pierresteguy.  */
 
 /* On 4.3 these lose if they come after xterm.h.  */
 /* On HP-UX 8.0 signal.h loses if it comes after config.h.  */
 /* Putting these at the beginning seems to be standard for other .c files.  */
-#include <stdio.h>
 #include <signal.h>
 
-#include "config.h"
+#include <config.h>
+
+#include <stdio.h>
+
+/* Need syssignal.h for various externs and definitions that may be required
+   by some configurations for calls to signal later in this source file.  */
+#include "syssignal.h"
 
 #ifdef HAVE_X_WINDOWS
 
@@ -55,30 +52,14 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 #ifdef BSD
 #include <sys/ioctl.h>
-#include <strings.h>
-#else /* ! defined (BSD) */
-#ifndef VMS
-#include <sys/termio.h>
-#include <string.h>
-#endif
 #endif /* ! defined (BSD) */
 
-/* Allow m- file to inhibit use of FIONREAD.  */
-#ifdef BROKEN_FIONREAD
-#undef FIONREAD
-#endif /* ! defined (BROKEN_FIONREAD) */
-
-/* We are unable to use interrupts if FIONREAD is not available,
-   so flush SIGIO so we won't try.  */
-#ifndef FIONREAD
-#ifdef SIGIO
-#undef SIGIO
-#endif /* ! defined (SIGIO) */
-#endif /* FIONREAD */
-
+#include "systty.h"
 #include "systime.h"
 
+#ifndef INCLUDED_FCNTL
 #include <fcntl.h>
+#endif
 #include <ctype.h>
 #include <errno.h>
 #include <setjmp.h>
@@ -99,6 +80,24 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #include "buffer.h"
 #include "window.h"
 
+#ifdef USE_X_TOOLKIT
+extern XtAppContext Xt_app_con;
+extern Widget Xt_app_shell;
+extern void free_frame_menubar ();
+extern void _XEditResCheckMessages ();
+#endif /* USE_X_TOOLKIT */
+
+#ifndef USE_X_TOOLKIT
+#define x_any_window_to_frame x_window_to_frame
+#define x_top_window_to_frame x_window_to_frame
+#endif
+
+#ifdef USE_X_TOOLKIT
+#ifndef XtNinitialState
+#define XtNinitialState "initialState"
+#endif
+#endif
+
 #ifdef HAVE_X11
 #define XMapWindow XMapRaised          /* Raise them when mapping. */
 #else /* ! defined (HAVE_X11) */
@@ -106,12 +105,33 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 /*#include <X/Xproto.h>        */
 #endif /* ! defined (HAVE_X11) */
 
+#ifdef FD_SET
+/* We could get this from param.h, but better not to depend on finding that.
+   And better not to risk that it might define other symbols used in this
+   file.  */
+#ifdef FD_SETSIZE
+#define MAXDESC FD_SETSIZE
+#else
+#define MAXDESC 64
+#endif
+#define SELECT_TYPE fd_set
+#else /* no FD_SET */
+#define MAXDESC 32
+#define SELECT_TYPE int
+
+/* Define the macros to access a single-int bitmap of descriptors.  */
+#define FD_SET(n, p) (*(p) |= (1 << (n)))
+#define FD_CLR(n, p) (*(p) &= ~(1 << (n)))
+#define FD_ISSET(n, p) (*(p) & (1 << (n)))
+#define FD_ZERO(p) (*(p) = 0)
+#endif /* no FD_SET */
+
 /* For sending Meta-characters.  Do we need this? */
 #define METABIT 0200
 
 #define min(a,b) ((a)<(b) ? (a) : (b))
 #define max(a,b) ((a)>(b) ? (a) : (b))
-
+\f
 /* Nonzero means we must reprint all windows
    because 1) we received an ExposeWindow event
    or 2) we received too many ExposeRegion events to record.
@@ -129,7 +149,7 @@ static int expose_all_icons;
 
 static struct event_queue x_expose_queue;
 
-/* ButtonPressed and ButtonReleased events, when received,
+/* ButtonPress and ButtonReleased events, when received,
    are copied into this queue for later processing.  */
 
 struct event_queue x_mouse_queue;
@@ -151,8 +171,12 @@ static FONT_TYPE *icon_font_info;
 
 /* Stuff for dealing with the main icon title. */
 
-extern Lisp_Object Vcommand_line_args;
-char *hostname, *x_id_name;
+extern Lisp_Object Vcommand_line_args, Vsystem_name;
+char *x_id_name;
+
+/* Initial values of argv and argc.  */
+extern char **initial_argv;
+extern int initial_argc;
 
 /* This is the X connection that we are using.  */
 
@@ -175,6 +199,9 @@ extern struct frame *updating_frame;
    x_focus_event_frame.  */
 struct frame *x_focus_frame;
 
+/* This is a frame waiting to be autoraised, within XTread_socket.  */
+struct frame *pending_autoraise_frame;
+
 /* The last frame mentioned in a FocusIn or FocusOut event.  This is
    separate from x_focus_frame, because whether or not LeaveNotify
    events cause us to lose focus depends on whether or not we have
@@ -207,6 +234,75 @@ static int highlight;
 static int curs_x;
 static int curs_y;
 
+/* Reusable Graphics Context for drawing a cursor in a non-default face. */
+static GC scratch_cursor_gc;
+
+/* Mouse movement.
+
+   In order to avoid asking for motion events and then throwing most
+   of them away or busy-polling the server for mouse positions, we ask
+   the server for pointer motion hints.  This means that we get only
+   one event per group of mouse movements.  "Groups" are delimited by
+   other kinds of events (focus changes and button clicks, for
+   example), or by XQueryPointer calls; when one of these happens, we
+   get another MotionNotify event the next time the mouse moves.  This
+   is at least as efficient as getting motion events when mouse
+   tracking is on, and I suspect only negligibly worse when tracking
+   is off.
+
+   The silly O'Reilly & Associates Nutshell guides barely document
+   pointer motion hints at all (I think you have to infer how they
+   work from an example), and the description of XQueryPointer doesn't
+   mention that calling it causes you to get another motion hint from
+   the server, which is very important.  */
+
+/* Where the mouse was last time we reported a mouse event.  */
+static FRAME_PTR last_mouse_frame;
+static XRectangle last_mouse_glyph;
+
+/* The scroll bar in which the last X motion event occurred.
+
+   If the last X motion event occurred in a scroll bar, we set this
+   so XTmouse_position can know whether to report a scroll bar motion or
+   an ordinary motion.
+
+   If the last X motion event didn't occur in a scroll bar, we set this
+   to Qnil, to tell XTmouse_position to return an ordinary motion event.  */
+static Lisp_Object last_mouse_scroll_bar;
+
+/* Record which buttons are currently pressed.  */
+unsigned int x_mouse_grabbed;
+
+/* This is a hack.  We would really prefer that XTmouse_position would
+   return the time associated with the position it returns, but there
+   doesn't seem to be any way to wrest the timestamp from the server
+   along with the position query.  So, we just keep track of the time
+   of the last movement we received, and return that in hopes that
+   it's somewhat accurate.  */
+static Time last_mouse_movement_time;
+
+/* These variables describe the range of text currently shown
+   in its mouse-face, together with the window they apply to.
+   As long as the mouse stays within this range, we need not
+   redraw anything on its account.  */
+static int mouse_face_beg_row, mouse_face_beg_col;
+static int mouse_face_end_row, mouse_face_end_col;
+static int mouse_face_past_end;
+static Lisp_Object mouse_face_window;
+static int mouse_face_face_id;
+
+/* 1 if a mouse motion event came and we didn't handle it right away because
+   gc was in progress.  */
+static int mouse_face_deferred_gc;
+
+/* FRAME and X, Y position of mouse when last checked for
+   highlighting.  X and Y can be negative or out of range for the frame.  */
+static FRAME_PTR mouse_face_mouse_frame;
+static int mouse_face_mouse_x, mouse_face_mouse_y;
+
+/* Nonzero means defer mouse-motion highlighting.  */
+static int mouse_face_defer;
+
 #ifdef HAVE_X11
 /* `t' if a mouse button is depressed. */
 
@@ -225,6 +321,8 @@ extern Window requestor_window;
 /* Nonzero enables some debugging for the X interface code. */
 extern int _Xdebug;
 
+extern Lisp_Object Qface, Qmouse_face;
+
 #else /* ! defined (HAVE_X11) */
 
 /* Bit patterns for the mouse cursor.  */
@@ -247,6 +345,11 @@ static short grey_bits[] = {
 static Pixmap GreyPixmap = 0;
 #endif /* ! defined (HAVE_X11) */
 
+#ifdef X_IO_BUG
+static int x_noop_count;
+#endif
+
+
 /* From time to time we get info on an Emacs window, here.  */
 
 static WINDOWINFO_TYPE windowinfo;
@@ -263,6 +366,14 @@ extern Cursor XCreateCursor ();
 extern FONT_TYPE *XOpenFont ();
 
 static void flashback ();
+static void redraw_previous_char ();
+static void redraw_following_char ();
+static unsigned int x_x_to_emacs_modifiers ();
+
+static int fast_find_position ();
+static void note_mouse_highlight ();
+static void clear_mouse_face ();
+static void show_mouse_face ();
 
 #ifndef HAVE_X11
 static void dumpqueue ();
@@ -280,7 +391,7 @@ static int XTclear_end_of_line ();
    of the frame being updated, so that the XT... functions do not
    need to take a frame as argument.  Most of the XT... functions
    should never be called except during an update, the only exceptions
-   being XTcursor_to, XTwrite_char and XTreassert_line_highlight.  */
+   being XTcursor_to, XTwrite_glyphs and XTreassert_line_highlight.  */
 
 extern int mouse_track_top, mouse_track_left, mouse_track_width;
 
@@ -297,6 +408,38 @@ XTupdate_begin (f)
   highlight = 0;
 
   BLOCK_INPUT;
+
+  if (f == mouse_face_mouse_frame)
+    {
+      /* Don't do highlighting for mouse motion during the update.  */
+      mouse_face_defer = 1;
+      if (!NILP (mouse_face_window))
+       {
+         int firstline, lastline, i;
+         struct window *w = XWINDOW (mouse_face_window);
+
+         /* Find the first, and the last+1, lines affected by redisplay.  */
+         for (firstline = 0; firstline < f->height; firstline++)
+           if (FRAME_DESIRED_GLYPHS (f)->enable[firstline])
+             break;
+
+         lastline = f->height;
+         for (i = f->height - 1; i >= 0; i--)
+           {
+             if (FRAME_DESIRED_GLYPHS (f)->enable[i])
+               break;
+             else
+               lastline = i;
+           }
+
+         /* Can we tell that this update does not affect the window
+            where the mouse highlight is?  If so, no need to turn off.  */
+         if (! (firstline > (XFASTINT (w->top) + window_internal_height (w))
+                || lastline < XFASTINT (w->top)))
+           /* Otherwise turn off the mouse highlight now.  */
+           clear_mouse_face ();
+       }
+    }
 #ifndef HAVE_X11
   dumpqueue ();
 #endif /* HAVE_X11 */
@@ -313,10 +456,6 @@ XTupdate_end (f)
 {      
   int mask;
 
-  if (updating_frame == 0
-      || updating_frame != f)
-    abort ();
-
   BLOCK_INPUT;
 #ifndef HAVE_X11
   dumpqueue ();
@@ -325,9 +464,36 @@ XTupdate_end (f)
 
   x_display_cursor (f, 1);
 
+  if (f == mouse_face_mouse_frame)
+    mouse_face_defer = 0;
+#if 0
+  /* This fails in the case of having updated only the echo area
+     if we have switched buffers.  In that case, FRAME_CURRENT_GLYPHS
+     has no relation to the current contents, and its charstarts
+     have no relation to the contents of the window-buffer.
+  I don't know a clean way to check
+     for that case.  window_end_valid isn't set up yet.  */
+  if (f == mouse_face_mouse_frame)
+    note_mouse_highlight (f, mouse_face_mouse_x, mouse_face_mouse_y);
+#endif
+
   XFlushQueue ();
   UNBLOCK_INPUT;
 }
+
+/* This is called after a redisplay on frame F.  */
+
+static
+XTframe_up_to_date (f)
+     FRAME_PTR f;
+{
+  if (mouse_face_deferred_gc || f == mouse_face_mouse_frame)
+    {
+      note_mouse_highlight (mouse_face_mouse_frame,
+                           mouse_face_mouse_x, mouse_face_mouse_y);
+      mouse_face_deferred_gc = 0;
+    }
+}
 \f
 /* External interface to control of standout mode.
    Call this when about to modify line at position VPOS
@@ -395,26 +561,31 @@ XTcursor_to (row, col)
 \f
 /* Display a sequence of N glyphs found at GP.
    WINDOW is the x-window to output to.  LEFT and TOP are starting coords.
-   HL is 1 if this text is highlighted, 2 if the cursor is on it.
+   HL is 1 if this text is highlighted, 2 if the cursor is on it,
+   3 if should appear in its mouse-face.
+   JUST_FOREGROUND if 1 means draw only the foreground;
+   don't alter the background.
 
    FONT is the default font to use (for glyphs whose font-code is 0).
 
    Since the display generation code is responsible for calling
    compute_char_face and compute_glyph_face on everything it puts in
    the display structure, we can assume that the face code on each
-   glyph is a valid index into FRAME_FACES (f), and the one to which
-   we can actually apply intern_face.  */
+   glyph is a valid index into FRAME_COMPUTED_FACES (f), and the one
+   to which we can actually apply intern_face.
+   Call this function with input blocked.  */
 
 #if 1
 /* This is the multi-face code.  */
 
 static void
-dumpglyphs (f, left, top, gp, n, hl)
+dumpglyphs (f, left, top, gp, n, hl, just_foreground)
      struct frame *f;
      int left, top;
      register GLYPH *gp; /* Points to first GLYPH. */
      register int n;  /* Number of glyphs to display. */
      int hl;
+     int just_foreground;
 {
   /* Holds characters to be displayed. */
   char *buf = (char *) alloca (f->width * sizeof (*buf));
@@ -422,6 +593,7 @@ dumpglyphs (f, left, top, gp, n, hl)
   register int tlen = GLYPH_TABLE_LENGTH;
   register Lisp_Object *tbase = GLYPH_TABLE_BASE;
   Window window = FRAME_X_WINDOW (f);
+  int orig_left = left;
 
   while (n > 0)
     {
@@ -430,7 +602,7 @@ dumpglyphs (f, left, top, gp, n, hl)
       int g = *gp;
 
       GLYPH_FOLLOW_ALIASES (tbase, tlen, g);
-      cf = GLYPH_FACE (g);
+      cf = FAST_GLYPH_FACE (g);
 
       /* Find the run of consecutive glyphs with the same face-code.
         Extract their character codes into BUF.  */
@@ -439,10 +611,10 @@ dumpglyphs (f, left, top, gp, n, hl)
        {
          g = *gp;
          GLYPH_FOLLOW_ALIASES (tbase, tlen, g);
-         if (GLYPH_FACE (g) != cf)
+         if (FAST_GLYPH_FACE (g) != cf)
            break;
 
-         *cp++ = GLYPH_CHAR (g);
+         *cp++ = FAST_GLYPH_CHAR (g);
          --n;
          ++gp;
        }
@@ -456,24 +628,26 @@ dumpglyphs (f, left, top, gp, n, hl)
        struct face *face = FRAME_DEFAULT_FACE (f);
        FONT_TYPE *font = FACE_FONT (face);
        GC gc = FACE_GC (face);
-       int defaulted = 1;
-       int gc_temporary = 0;
+
+       /* HL = 3 means use a mouse face previously chosen.  */
+       if (hl == 3)
+         cf = mouse_face_face_id;
 
        /* First look at the face of the text itself.  */
        if (cf != 0)
          {
-           /* The face codes on the glyphs must be valid indices into the
-              frame's face table.  */
-           if (cf < 0 || cf >= FRAME_N_FACES (f))
-             abort ();
+           /* It's possible for the display table to specify
+              a face code that is out of range.  Use 0 in that case.  */
+           if (cf < 0 || cf >= FRAME_N_COMPUTED_FACES (f)
+               || FRAME_COMPUTED_FACES (f) [cf] == 0)
+             cf = 0;
 
            if (cf == 1)
              face = FRAME_MODE_LINE_FACE (f);
            else
-             face = intern_face (f, FRAME_FACES (f) [cf]);
+             face = intern_face (f, FRAME_COMPUTED_FACES (f) [cf]);
            font = FACE_FONT (face);
            gc = FACE_GC (face);
-           defaulted = 0;
          }
 
        /* Then comes the distinction between modeline and normal text.  */
@@ -484,7 +658,6 @@ dumpglyphs (f, left, top, gp, n, hl)
            face = FRAME_MODE_LINE_FACE (f);
            font = FACE_FONT (face);
            gc   = FACE_GC   (face);
-           defaulted = 0;
          }
 
 #define FACE_DEFAULT (~0)
@@ -492,9 +665,11 @@ dumpglyphs (f, left, top, gp, n, hl)
        /* Now override that if the cursor's on this character.  */
        if (hl == 2)
          {
-           if (defaulted
-               || !face->font
-               || (int) face->font == FACE_DEFAULT)
+           if ((!face->font
+                || (int) face->font == FACE_DEFAULT
+                || face->font == f->display.x->font)
+               && face->background == f->display.x->background_pixel
+               && face->foreground == f->display.x->foreground_pixel)
              {
                gc = f->display.x->cursor_gc;
              }
@@ -505,28 +680,73 @@ dumpglyphs (f, left, top, gp, n, hl)
                unsigned long mask;
 
                xgcv.background = f->display.x->cursor_pixel;
-               xgcv.foreground = f->display.x->cursor_foreground_pixel;
+               xgcv.foreground = face->background;
+               /* If the glyph would be invisible,
+                  try a different foreground.  */
+               if (xgcv.foreground == xgcv.background)
+                 xgcv.foreground = face->foreground;
+               if (xgcv.foreground == xgcv.background)
+                 xgcv.foreground = f->display.x->cursor_foreground_pixel;
+               if (xgcv.foreground == xgcv.background)
+                 xgcv.foreground = face->foreground;
+               /* Make sure the cursor is distinct from text in this face.  */
+               if (xgcv.background == face->background
+                   && xgcv.foreground == face->foreground)
+                 {
+                   xgcv.background = face->foreground;
+                   xgcv.foreground = face->background;
+                 }
                xgcv.font = face->font->fid;
                xgcv.graphics_exposures = 0;
                mask = GCForeground | GCBackground | GCFont | GCGraphicsExposures;
-               gc = XCreateGC (x_current_display, FRAME_X_WINDOW (f),
-                               mask, &xgcv);
+               if (scratch_cursor_gc)
+                 XChangeGC (x_current_display, scratch_cursor_gc, mask, &xgcv);
+               else
+                 scratch_cursor_gc =
+                   XCreateGC (x_current_display, window, mask, &xgcv);
+               gc = scratch_cursor_gc;
 #if 0
+/* If this code is restored, it must also reset to the default stipple
+   if necessary. */
                if (face->stipple && face->stipple != FACE_DEFAULT)
                  XSetStipple (x_current_display, gc, face->stipple);
 #endif
-               gc_temporary = 1;
              }
          }
 
        if ((int) font == FACE_DEFAULT)
          font = f->display.x->font;
 
-       XDrawImageString (x_current_display, window, gc,
-                         left, top + FONT_BASE (font), buf, len);
+       if (just_foreground)
+         XDrawString (x_current_display, window, gc,
+                      left, top + FONT_BASE (font), buf, len);
+       else
+         {
+           XDrawImageString (x_current_display, window, gc,
+                             left, top + FONT_BASE (font), buf, len);
+           /* Clear the rest of the line's height.  */
+           if (f->display.x->line_height != FONT_HEIGHT (font))
+             XClearArea (x_current_display, window, left,
+                         top + FONT_HEIGHT (font),
+                         FONT_WIDTH (font) * len,
+                         /* This is how many pixels of height
+                            we have to clear.  */
+                         f->display.x->line_height - FONT_HEIGHT (font),
+                         False);
+         }
 
-       if (gc_temporary)
-         XFreeGC (x_current_display, gc);
+#if 0 /* Doesn't work, because it uses FRAME_CURRENT_GLYPHS,
+        which often is not up to date yet.  */
+       if (!just_foreground)
+         {
+           if (left == orig_left)
+             redraw_previous_char (f, PIXEL_TO_CHAR_COL (f, left),
+                                   PIXEL_TO_CHAR_ROW (f, top), hl == 1);
+           if (n == 0)
+             redraw_following_char (f, PIXEL_TO_CHAR_COL (f, left + len * FONT_WIDTH (font)),
+                                    PIXEL_TO_CHAR_ROW (f, top), hl == 1);
+         }
+#endif
 
        /* We should probably check for XA_UNDERLINE_POSITION and
           XA_UNDERLINE_THICKNESS properties on the font, but let's
@@ -614,7 +834,7 @@ XTwrite_glyphs (start, len)
   dumpglyphs (f,
              CHAR_TO_PIXEL_COL (f, curs_x),
              CHAR_TO_PIXEL_ROW (f, curs_y),
-             start, len, highlight);
+             start, len, highlight, 0);
 
   /* If we drew on top of the cursor, note that it is turned off.  */
   if (curs_y == f->phys_cursor_y
@@ -670,14 +890,16 @@ XTclear_end_of_line (first_unused)
              CHAR_TO_PIXEL_COL (f, curs_x),
              CHAR_TO_PIXEL_ROW (f, curs_y),
              FONT_WIDTH (f->display.x->font) * (first_unused - curs_x),
-             FONT_HEIGHT (f->display.x->font), False);
-             
+             f->display.x->line_height, False);
+#if 0
+  redraw_previous_char (f, curs_x, curs_y, highlight);
+#endif
 #else /* ! defined (HAVE_X11) */
   XPixSet (FRAME_X_WINDOW (f),
           CHAR_TO_PIXEL_COL (f, curs_x),
           CHAR_TO_PIXEL_ROW (f, curs_y),
           FONT_WIDTH (f->display.x->font) * (first_unused - curs_x),
-          FONT_HEIGHT (f->display.x->font),
+          f->display.x->line_height,
           f->display.x->background_pixel);     
 #endif /* ! defined (HAVE_X11) */
 
@@ -713,10 +935,183 @@ XTclear_frame ()
   UNBLOCK_INPUT;
 }
 \f
+#if 0
+/* This currently does not work because FRAME_CURRENT_GLYPHS doesn't
+   always contain the right glyphs to use.
+
+   It also needs to be changed to look at the details of the font and
+   see whether there is really overlap, and do nothing when there is
+   not.  This can use font_char_overlap_left and font_char_overlap_right,
+   but just how to use them is not clear.  */
+
+/* Erase the character (if any) at the position just before X, Y in frame F,
+   then redraw it and the character before it.
+   This is necessary when we erase starting at X,
+   in case the character after X overlaps into the one before X.
+   Call this function with input blocked.  */
+
+static void
+redraw_previous_char (f, x, y, highlight_flag)
+     FRAME_PTR f;
+     int x, y;
+     int highlight_flag;
+{
+  /* Erase the character before the new ones, in case
+     what was here before overlaps it.
+     Reoutput that character, and the previous character
+     (in case the previous character overlaps it).  */
+  if (x > 0)
+    {
+      int start_x = x - 2;
+      if (start_x < 0)
+       start_x = 0;
+      XClearArea (x_current_display, FRAME_X_WINDOW (f),
+                 CHAR_TO_PIXEL_COL (f, x - 1),
+                 CHAR_TO_PIXEL_ROW (f, y),
+                 FONT_WIDTH (f->display.x->font),
+                 f->display.x->line_height, False);
+
+      dumpglyphs (f, CHAR_TO_PIXEL_COL (f, start_x),
+                 CHAR_TO_PIXEL_ROW (f, y),
+                 &FRAME_CURRENT_GLYPHS (f)->glyphs[y][start_x],
+                 x - start_x, highlight_flag, 1);
+    }
+}
+
+/* Erase the character (if any) at the position X, Y in frame F,
+   then redraw it and the character after it.
+   This is necessary when we erase endng at X,
+   in case the character after X overlaps into the one before X.
+   Call this function with input blocked.  */
+
+static void
+redraw_following_char (f, x, y, highlight_flag)
+     FRAME_PTR f;
+     int x, y;
+     int highlight_flag;
+{
+  int limit = FRAME_CURRENT_GLYPHS (f)->used[y];
+  /* Erase the character after the new ones, in case
+     what was here before overlaps it.
+     Reoutput that character, and the following character
+     (in case the following character overlaps it).  */
+  if (x < limit
+      && FRAME_CURRENT_GLYPHS (f)->glyphs[y][x] != SPACEGLYPH)
+    {
+      int end_x = x + 2;
+      if (end_x > limit)
+       end_x = limit;
+      XClearArea (x_current_display, FRAME_X_WINDOW (f),
+                 CHAR_TO_PIXEL_COL (f, x),
+                 CHAR_TO_PIXEL_ROW (f, y),
+                 FONT_WIDTH (f->display.x->font),
+                 f->display.x->line_height, False);
+
+      dumpglyphs (f, CHAR_TO_PIXEL_COL (f, x),
+                 CHAR_TO_PIXEL_ROW (f, y),
+                 &FRAME_CURRENT_GLYPHS (f)->glyphs[y][x],
+                 end_x - x, highlight_flag, 1);
+    }
+}
+#endif /* 0 */
+\f
+#if 0 /* Not in use yet */
+
+/* Return 1 if character C in font F extends past its left edge.  */
+
+static int
+font_char_overlap_left (font, c)
+     XFontStruct *font;
+     int c;
+{
+  XCharStruct *s;
+
+  /* Find the bounding-box info for C.  */
+  if (font->per_char == 0)
+    s = &font->max_bounds;
+  else
+    {
+      int rowlen = font->max_char_or_byte2 - font->min_char_or_byte2 + 1;
+      int row, within;
+       
+      /* Decode char into row number (byte 1) and code within row (byte 2).  */
+      row = c >> 8;
+      within = c & 0177;
+      if (!(within >= font->min_char_or_byte2
+           && within <= font->max_char_or_byte2
+           && row >= font->min_byte1
+           && row <= font->max_byte1))
+       {
+         /* If char is out of range, try the font's default char instead.  */
+         c = font->default_char;
+         row = c >> (INTBITS - 8);
+         within = c & 0177;
+       }
+      if (!(within >= font->min_char_or_byte2
+           && within <= font->max_char_or_byte2
+           && row >= font->min_byte1
+           && row <= font->max_byte1))
+       /* Still out of range means this char does not overlap.  */
+       return 0;
+      else
+       /* We found the info for this char.  */
+       s = (font->per_char + (within - font->min_char_or_byte2)
+            + row * rowlen);
+    }
+
+  return (s && s->lbearing < 0);
+}
+
+/* Return 1 if character C in font F extends past its right edge.  */
+
+static int
+font_char_overlap_right (font, c)
+     XFontStruct *font;
+     int c;
+{
+  XCharStruct *s;
+
+  /* Find the bounding-box info for C.  */
+  if (font->per_char == 0)
+    s = &font->max_bounds;
+  else
+    {
+      int rowlen = font->max_char_or_byte2 - font->min_char_or_byte2 + 1;
+      int row, within;
+       
+      /* Decode char into row number (byte 1) and code within row (byte 2).  */
+      row = c >> 8;
+      within = c & 0177;
+      if (!(within >= font->min_char_or_byte2
+           && within <= font->max_char_or_byte2
+           && row >= font->min_byte1
+           && row <= font->max_byte1))
+       {
+         /* If char is out of range, try the font's default char instead.  */
+         c = font->default_char;
+         row = c >> (INTBITS - 8);
+         within = c & 0177;
+       }
+      if (!(within >= font->min_char_or_byte2
+           && within <= font->max_char_or_byte2
+           && row >= font->min_byte1
+           && row <= font->max_byte1))
+       /* Still out of range means this char does not overlap.  */
+       return 0;
+      else
+       /* We found the info for this char.  */
+       s = (font->per_char + (within - font->min_char_or_byte2)
+            + row * rowlen);
+    }
+
+  return (s && s->rbearing >= s->width);
+}
+#endif /* 0 */
+\f
 /* Invert the middle quarter of the frame for .15 sec.  */
 
 /* We use the select system call to do the waiting, so we have to make sure
-   it's avaliable.  If it isn't, we just won't do visual bells.  */
+   it's available.  If it isn't, we just won't do visual bells.  */
 #if defined (HAVE_TIMEVAL) && defined (HAVE_SELECT)
 
 /* Subtract the `struct timeval' values X and Y,
@@ -783,7 +1178,7 @@ XTflash (f)
       {
        struct timeval wakeup, now;
 
-       gettimeofday (&wakeup, (struct timezone *) 0);
+       EMACS_GET_TIME (wakeup);
 
        /* Compute time to wait until, propagating carry from usecs.  */
        wakeup.tv_usec += 150000;
@@ -795,7 +1190,7 @@ XTflash (f)
          {
            struct timeval timeout;
 
-           gettimeofday (&timeout, (struct timezone *)0);
+           EMACS_GET_TIME (timeout);
 
            /* In effect, timeout = wakeup - timeout.
               Break if result would be negative.  */
@@ -823,9 +1218,9 @@ XTflash (f)
 /* Make audible bell.  */
 
 #ifdef HAVE_X11
-#define XRINGBELL XBell(x_current_display, 0)
+#define XRINGBELL XBell (x_current_display, 0)
 #else /* ! defined (HAVE_X11) */
-#define XRINGBELL XFeep(0);
+#define XRINGBELL XFeep (0);
 #endif /* ! defined (HAVE_X11) */
 
 XTring_bell ()
@@ -914,14 +1309,14 @@ stufflines (n)
                 FRAME_X_WINDOW (f), f->display.x->normal_gc,
                 intborder, CHAR_TO_PIXEL_ROW (f, topregion),
                 f->width * FONT_WIDTH (f->display.x->font),
-                length * FONT_HEIGHT (f->display.x->font), intborder,
+                length * f->display.x->line_height, intborder,
                 CHAR_TO_PIXEL_ROW (f, newtop));
 #else /* ! defined (HAVE_X11) */
       XMoveArea (FRAME_X_WINDOW (f),
                 intborder, CHAR_TO_PIXEL_ROW (f, topregion),
                 intborder, CHAR_TO_PIXEL_ROW (f, newtop),
                 f->width * FONT_WIDTH (f->display.x->font),
-                length * FONT_HEIGHT (f->display.x->font));
+                length * f->display.x->line_height);
       /* Now we must process any ExposeRegion events that occur
         if the area being copied from is obscured.
         We can't let it wait because further i/d operations
@@ -938,13 +1333,13 @@ stufflines (n)
       XClearArea (x_current_display, FRAME_X_WINDOW (f), intborder, 
                  CHAR_TO_PIXEL_ROW (f, topregion),
                  f->width * FONT_WIDTH (f->display.x->font),
-                 n * FONT_HEIGHT (f->display.x->font), False);
+                 n * f->display.x->line_height, False);
 #else /* ! defined (HAVE_X11) */
       XPixSet (FRAME_X_WINDOW (f),
               intborder,
               CHAR_TO_PIXEL_ROW (f, topregion),
               f->width * FONT_WIDTH (f->display.x->font),
-              n * FONT_HEIGHT (f->display.x->font),
+              n * f->display.x->line_height,
               f->display.x->background_pixel);
 #endif /* ! defined (HAVE_X11) */
     }
@@ -976,12 +1371,12 @@ scraplines (n)
          XClearArea (x_current_display, FRAME_X_WINDOW (f), intborder,
                      CHAR_TO_PIXEL_ROW (f, curs_y),
                      f->width * FONT_WIDTH (f->display.x->font),
-                     (flexlines - curs_y) * FONT_HEIGHT (f->display.x->font), False);
+                     (flexlines - curs_y) * f->display.x->line_height, False);
 #else /* ! defined (HAVE_X11) */
          XPixSet (FRAME_X_WINDOW (f),
                   intborder, CHAR_TO_PIXEL_ROW (f, curs_y),
                   f->width * FONT_WIDTH (f->display.x->font),
-                  (flexlines - curs_y) * FONT_HEIGHT (f->display.x->font),
+                  (flexlines - curs_y) * f->display.x->line_height,
                   f->display.x->background_pixel);
 #endif /* ! defined (HAVE_X11) */
        }
@@ -994,20 +1389,20 @@ scraplines (n)
                 intborder,
                 CHAR_TO_PIXEL_ROW (f, curs_y + n),
                 f->width * FONT_WIDTH (f->display.x->font),
-                (flexlines - (curs_y + n)) * FONT_HEIGHT (f->display.x->font),
+                (flexlines - (curs_y + n)) * f->display.x->line_height,
                 intborder, CHAR_TO_PIXEL_ROW (f, curs_y));
       XClearArea (x_current_display, FRAME_X_WINDOW (f),
                  intborder,
                  CHAR_TO_PIXEL_ROW (f, flexlines - n),
                  f->width * FONT_WIDTH (f->display.x->font),
-                 n * FONT_HEIGHT (f->display.x->font), False);
+                 n * f->display.x->line_height, False);
 #else /* ! defined (HAVE_X11) */
       XMoveArea (FRAME_X_WINDOW (f),
                 intborder,
                 CHAR_TO_PIXEL_ROW (f, curs_y + n),
                 intborder, CHAR_TO_PIXEL_ROW (f, curs_y),
                 f->width * FONT_WIDTH (f->display.x->font),
-                (flexlines - (curs_y + n)) * FONT_HEIGHT (f->display.x->font));
+                (flexlines - (curs_y + n)) * f->display.x->line_height);
       /* Now we must process any ExposeRegion events that occur
         if the area being copied from is obscured.
         We can't let it wait because further i/d operations
@@ -1016,7 +1411,7 @@ scraplines (n)
       XPixSet (FRAME_X_WINDOW (f), intborder,
               CHAR_TO_PIXEL_ROW (f, flexlines - n),
               f->width * FONT_WIDTH (f->display.x->font),
-              n * FONT_HEIGHT (f->display.x->font), f->display.x->background_pixel);
+              n * f->display.x->line_height, f->display.x->background_pixel);
 #endif /* ! defined (HAVE_X11) */
     }
 }
@@ -1050,7 +1445,8 @@ static void clear_cursor ();
 /* Output into a rectangle of an X-window (for frame F)
    the characters in f->phys_lines that overlap that rectangle.
    TOP and LEFT are the position of the upper left corner of the rectangle.
-   ROWS and COLS are the size of the rectangle.  */
+   ROWS and COLS are the size of the rectangle.
+   Call this function with input blocked.  */
 
 static void
 dumprectangle (f, left, top, cols, rows)
@@ -1076,17 +1472,17 @@ dumprectangle (f, left, top, cols, rows)
     /* If the rectangle includes any of the internal border area,
        redisplay the border emphasis.  */
     if (top < intborder || left < intborder
-       || bottom > intborder + f->height * FONT_HEIGHT (f->display.x->font)
-       || right > intborder + f->width * FONT_WIDTH (f->display.x->font))
+       || bottom > intborder + f->height * f->display.x->line_height
+       || right > intborder + f->width * f->display.x->line_height)
       dumpborder (f, 0);
   }
-#endif /* HAVE_X11             Window manger does this for X11. */
+#endif /* not HAVE_X11         Window manger does this for X11. */
   
   /* Convert rectangle edges in pixels to edges in chars.
      Round down for left and top, up for right and bottom.  */
   top  = PIXEL_TO_CHAR_ROW (f, top);
   left = PIXEL_TO_CHAR_COL (f, left);
-  bottom += (FONT_HEIGHT (f->display.x->font) - 1);
+  bottom += (f->display.x->line_height - 1);
   right += (FONT_WIDTH (f->display.x->font) - 1);
   bottom = PIXEL_TO_CHAR_ROW (f, bottom);
   right = PIXEL_TO_CHAR_COL (f, right);
@@ -1131,7 +1527,7 @@ dumprectangle (f, left, top, cols, rows)
                  CHAR_TO_PIXEL_COL (f, left),
                  CHAR_TO_PIXEL_ROW (f, y),
                  line, min (cols, active_frame->used[y] - left),
-                 active_frame->highlight[y]);
+                 active_frame->highlight[y], 0);
     }
 
   /* Turn the cursor on if we turned it off.  */
@@ -1201,7 +1597,7 @@ x_do_pending_expose ()
                        / FONT_WIDTH (f->display.x->font));
          temp_height = ((windowinfo.height- 2 * intborder
                          - f->display.x->h_scroll_bar_height)
-                        / FONT_HEIGHT (f->display.x->font));
+                        / f->display.x->line_height);
          if (temp_width != f->width || temp_height != f->height)
            {
              change_frame_size (f, max (1, temp_height),
@@ -1236,9 +1632,14 @@ static void
 frame_highlight (frame)
      struct frame *frame;
 {
-  if (! EQ (Vx_no_window_manager, Qnil))
-    XSetWindowBorder (x_current_display, FRAME_X_WINDOW (frame),
-                     frame->display.x->border_pixel);
+  /* We used to only do this if Vx_no_window_manager was non-nil, but
+     the ICCCM (section 4.1.6) says that the window's border pixmap
+     and border pixel are window attributes which are "private to the
+     client", so we can always change it to whatever we want.  */
+  BLOCK_INPUT;
+  XSetWindowBorder (x_current_display, FRAME_X_WINDOW (frame),
+                   frame->display.x->border_pixel);
+  UNBLOCK_INPUT;
   x_display_cursor (frame, 1);
 }
 
@@ -1246,9 +1647,14 @@ static void
 frame_unhighlight (frame)
      struct frame *frame;
 {
-  if (! EQ (Vx_no_window_manager, Qnil))
-    XSetWindowBorderPixmap (x_current_display, FRAME_X_WINDOW (frame),
-                           frame->display.x->border_tile);
+  /* We used to only do this if Vx_no_window_manager was non-nil, but
+     the ICCCM (section 4.1.6) says that the window's border pixmap
+     and border pixel are window attributes which are "private to the
+     client", so we can always change it to whatever we want.  */
+  BLOCK_INPUT;
+  XSetWindowBorderPixmap (x_current_display, FRAME_X_WINDOW (frame),
+                         frame->display.x->border_tile);
+  UNBLOCK_INPUT;
   x_display_cursor (frame, 1);
 }
 #else /* ! defined (HAVE_X11) */
@@ -1301,7 +1707,7 @@ static void XTframe_rehighlight ();
 /* The focus has changed.  Update the frames as necessary to reflect
    the new situation.  Note that we can't change the selected frame
    here, because the lisp code we are interrupting might become confused.
-   Each event gets marked with the frame in which it occured, so the
+   Each event gets marked with the frame in which it occurred, so the
    lisp code can tell when the switch took place by examining the events.  */
 
 static void
@@ -1329,7 +1735,9 @@ x_new_focus_frame (frame)
 #endif /* ! 0 */
 
       if (x_focus_frame && x_focus_frame->auto_raise)
-       x_raise_frame (x_focus_frame);
+       pending_autoraise_frame = x_focus_frame;
+      else
+       pending_autoraise_frame = 0;
     }
 
   XTframe_rehighlight ();
@@ -1368,42 +1776,7 @@ XTframe_rehighlight ()
     }
 }
 \f
-/* Mouse clicks and mouse movement.  Rah.  */
-#ifdef HAVE_X11
-
-/* Given a pixel position (PIX_X, PIX_Y) on the frame F, return
-   glyph co-ordinates in (*X, *Y).  Set *BOUNDS to the rectangle
-   that the glyph at X, Y occupies, if BOUNDS != 0.  */
-static void
-pixel_to_glyph_coords (f, pix_x, pix_y, x, y, bounds)
-     FRAME_PTR f;
-     register unsigned int pix_x, pix_y;
-     register int *x, *y;
-     XRectangle *bounds;
-{
-  pix_x = PIXEL_TO_CHAR_COL (f, pix_x);
-  pix_y = PIXEL_TO_CHAR_ROW (f, pix_y);
-
-  if (bounds)
-    {
-      bounds->width  = FONT_WIDTH  (f->display.x->font);
-      bounds->height = FONT_HEIGHT (f->display.x->font);
-      bounds->x = CHAR_TO_PIXEL_COL (f, pix_x);
-      bounds->y = CHAR_TO_PIXEL_ROW (f, pix_y);
-    }
-
-  if (pix_x < 0) pix_x = 0;
-  else if (pix_x > f->width) pix_x = f->width;
-
-  if (pix_y < 0) pix_y = 0;
-  else if (pix_y > f->height) pix_y = f->height;
-
-  *x = pix_x;
-  *y = pix_y;
-}
-
-/* Any buttons grabbed. */
-unsigned int x_mouse_grabbed;
+/* Keyboard processing - modifier keys, vendor-specific keysyms, etc. */
 
 /* Which modifier keys are on which modifier bits?
 
@@ -1467,6 +1840,10 @@ x_find_modifier_meanings ()
          KeyCode code =
            mods->modifiermap[(row * mods->max_keypermod) + col];
 
+         /* Zeroes are used for filler.  Skip them.  */
+         if (code == 0)
+           continue;
+
          /* Are any of this keycode's keysyms a meta key?  */
          {
            int code_col;
@@ -1526,7 +1903,6 @@ x_find_modifier_meanings ()
   XFreeModifiermap (mods);
 }
 
-
 /* Convert between the modifier bits X uses and the modifier bits
    Emacs uses.  */
 static unsigned int
@@ -1552,6 +1928,67 @@ x_emacs_to_x_modifiers (state)
          | ((state & ctrl_modifier)            ? ControlMask      : 0)
          | ((state & meta_modifier)            ? x_meta_mod_mask  : 0));
 }
+\f
+/* Mouse clicks and mouse movement.  Rah.  */
+#ifdef HAVE_X11
+
+/* Given a pixel position (PIX_X, PIX_Y) on the frame F, return
+   glyph co-ordinates in (*X, *Y).  Set *BOUNDS to the rectangle
+   that the glyph at X, Y occupies, if BOUNDS != 0.
+   If NOCLIP is nonzero, do not force the value into range.  */
+
+void
+pixel_to_glyph_coords (f, pix_x, pix_y, x, y, bounds, noclip)
+     FRAME_PTR f;
+     register int pix_x, pix_y;
+     register int *x, *y;
+     XRectangle *bounds;
+     int noclip;
+{
+  /* Arrange for the division in PIXEL_TO_CHAR_COL etc. to round down
+     even for negative values.  */
+  if (pix_x < 0)
+    pix_x -= FONT_WIDTH ((f)->display.x->font) - 1;
+  if (pix_y < 0)
+    pix_y -= (f)->display.x->line_height - 1;
+
+  pix_x = PIXEL_TO_CHAR_COL (f, pix_x);
+  pix_y = PIXEL_TO_CHAR_ROW (f, pix_y);
+
+  if (bounds)
+    {
+      bounds->width  = FONT_WIDTH  (f->display.x->font);
+      bounds->height = f->display.x->line_height;
+      bounds->x = CHAR_TO_PIXEL_COL (f, pix_x);
+      bounds->y = CHAR_TO_PIXEL_ROW (f, pix_y);
+    }
+
+  if (!noclip)
+    {
+      if (pix_x < 0)
+       pix_x = 0;
+      else if (pix_x > f->width)
+       pix_x = f->width;
+
+      if (pix_y < 0)
+       pix_y = 0;
+      else if (pix_y > f->height)
+       pix_y = f->height;
+    }
+
+  *x = pix_x;
+  *y = pix_y;
+}
+
+void
+glyph_to_pixel_coords (f, x, y, pix_x, pix_y)
+     FRAME_PTR f;
+     register int x, y;
+     register int *pix_x, *pix_y;
+{
+  *pix_x = CHAR_TO_PIXEL_COL (f, x);
+  *pix_y = CHAR_TO_PIXEL_ROW (f, y);
+}
 
 /* Prepare a mouse-event in *RESULT for placement in the input queue.
 
@@ -1567,79 +2004,50 @@ construct_mouse_click (result, event, f)
   /* Make the event type no_event; we'll change that when we decide
      otherwise.  */
   result->kind = mouse_click;
-  XSET (result->code, Lisp_Int, event->button - Button1);
+  result->code = event->button - Button1;
   result->timestamp = event->time;
   result->modifiers = (x_x_to_emacs_modifiers (event->state)
                       | (event->type == ButtonRelease
                          ? up_modifier 
                          : down_modifier));
 
-  /* Notice if the mouse is still grabbed.  */
-  if (event->type == ButtonPress)
-    {
-      if (! x_mouse_grabbed)
-       Vmouse_depressed = Qt;
-      x_mouse_grabbed |= (1 << event->button);
-    }
-  else if (event->type == ButtonRelease)
-    {
-      x_mouse_grabbed &= ~(1 << event->button);
-      if (!x_mouse_grabbed)
-       Vmouse_depressed = Qnil;
-    }
-
   {
     int row, column;
 
-    pixel_to_glyph_coords (f, event->x, event->y, &column, &row, NULL);
+#if 0
+    pixel_to_glyph_coords (f, event->x, event->y, &column, &row, NULL, 0);
     XFASTINT (result->x) = column;
     XFASTINT (result->y) = row;
+#endif
+    XSET (result->x, Lisp_Int, event->x);
+    XSET (result->y, Lisp_Int, event->y);
     XSET (result->frame_or_window, Lisp_Frame, f);
   }
 }
 
+/* Prepare a menu-event in *RESULT for placement in the input queue.  */
 
-/* Mouse movement.  Rah.
-
-   In order to avoid asking for motion events and then throwing most
-   of them away or busy-polling the server for mouse positions, we ask
-   the server for pointer motion hints.  This means that we get only
-   one event per group of mouse movements.  "Groups" are delimited by
-   other kinds of events (focus changes and button clicks, for
-   example), or by XQueryPointer calls; when one of these happens, we
-   get another MotionNotify event the next time the mouse moves.  This
-   is at least as efficient as getting motion events when mouse
-   tracking is on, and I suspect only negligibly worse when tracking
-   is off.
-
-   The silly O'Reilly & Associates Nutshell guides barely document
-   pointer motion hints at all (I think you have to infer how they
-   work from an example), and the description of XQueryPointer doesn't
-   mention that calling it causes you to get another motion hint from
-   the server, which is very important.  */
-
-/* Where the mouse was last time we reported a mouse event.  */
-static FRAME_PTR last_mouse_frame;
-static XRectangle last_mouse_glyph;
-
-/* The scroll bar in which the last X motion event occurred.
-
-   If the last X motion event occured in a scroll bar, we set this
-   so XTmouse_position can know whether to report a scroll bar motion or
-   an ordinary motion.
-
-   If the last X motion event didn't occur in a scroll bar, we set this
-   to Qnil, to tell XTmouse_position to return an ordinary motion event.  */
-static Lisp_Object last_mouse_scroll_bar;
-
-/* This is a hack.  We would really prefer that XTmouse_position would
-   return the time associated with the position it returns, but there
-   doesn't seem to be any way to wrest the timestamp from the server
-   along with the position query.  So, we just keep track of the time
-   of the last movement we received, and return that in hopes that
-   it's somewhat accurate.  */
-static Time last_mouse_movement_time;
+static Lisp_Object
+construct_menu_click (result, event, f)
+     struct input_event *result;
+     XButtonEvent *event;
+     struct frame *f;
+{
+  /* Make the event type no_event; we'll change that when we decide
+     otherwise.  */
+  result->kind = mouse_click;
+  XSET (result->code, Lisp_Int, event->button - Button1);
+  result->timestamp = event->time;
+  result->modifiers = (x_x_to_emacs_modifiers (event->state)
+                      | (event->type == ButtonRelease
+                         ? up_modifier 
+                         : down_modifier));
 
+  XSET (result->x, Lisp_Int, event->x);
+  XSET (result->y, Lisp_Int, -1);
+  XSET (result->frame_or_window, Lisp_Frame, f);
+}
+\f
 /* Function to report a mouse movement to the mainstream Emacs code.
    The input handler calls this.
 
@@ -1647,6 +2055,7 @@ static Time last_mouse_movement_time;
    If the mouse is over a different glyph than it was last time, tell
    the mainstream emacs code by setting mouse_moved.  If not, ask for
    another motion event, so we can check again the next time it moves.  */
+
 static void
 note_mouse_movement (frame, event)
      FRAME_PTR frame;
@@ -1655,14 +2064,46 @@ note_mouse_movement (frame, event)
 {
   last_mouse_movement_time = event->time;
 
+  if (event->window != FRAME_X_WINDOW (frame))
+    {
+      mouse_moved = 1;
+      last_mouse_scroll_bar = Qnil;
+
+      note_mouse_highlight (frame, -1, -1);
+
+      /* Ask for another mouse motion event.  */
+      {
+       int dummy;
+       Window dummy_window;
+
+       XQueryPointer (event->display, FRAME_X_WINDOW (frame),
+                      &dummy_window, &dummy_window,
+                      &dummy, &dummy, &dummy, &dummy,
+                      (unsigned int *) &dummy);
+      }
+    }
+
   /* Has the mouse moved off the glyph it was on at the last sighting?  */
-  if (event->x < last_mouse_glyph.x
-      || event->x >= last_mouse_glyph.x + last_mouse_glyph.width
-      || event->y < last_mouse_glyph.y
-      || event->y >= last_mouse_glyph.y + last_mouse_glyph.height)
+  else if (event->x < last_mouse_glyph.x
+          || event->x >= last_mouse_glyph.x + last_mouse_glyph.width
+          || event->y < last_mouse_glyph.y
+          || event->y >= last_mouse_glyph.y + last_mouse_glyph.height)
     {
       mouse_moved = 1;
       last_mouse_scroll_bar = Qnil;
+
+      note_mouse_highlight (frame, event->x, event->y);
+
+      /* Ask for another mouse motion event.  */
+      {
+       int dummy;
+       Window dummy_window;
+
+       XQueryPointer (event->display, FRAME_X_WINDOW (frame),
+                      &dummy_window, &dummy_window,
+                      &dummy, &dummy, &dummy, &dummy,
+                      (unsigned int *) &dummy);
+      }
     }
   else
     {
@@ -1670,14 +2111,330 @@ note_mouse_movement (frame, event)
         event the next time the mouse moves and we can see if it's
         *still* on the same glyph.  */
       int dummy;
+      Window dummy_window;
       
-      XQueryPointer (event->display, event->window,
-                    (Window *) &dummy, (Window *) &dummy,
+      XQueryPointer (event->display, FRAME_X_WINDOW (frame),
+                    &dummy_window, &dummy_window,
                     &dummy, &dummy, &dummy, &dummy,
                     (unsigned int *) &dummy);
     }
 }
 
+/* This is used for debugging, to turn off note_mouse_highlight.  */
+static int disable_mouse_highlight;
+
+/* Take proper action when the mouse has moved to position X, Y on frame F
+   as regards highlighting characters that have mouse-face properties.
+   Also dehighlighting chars where the mouse was before.
+   X and Y can be negative or out of range.  */
+
+static void
+note_mouse_highlight (f, x, y)
+     FRAME_PTR f;
+{
+  int row, column, portion;
+  XRectangle new_glyph;
+  Lisp_Object window;
+  struct window *w;
+
+  if (disable_mouse_highlight)
+    return;
+
+  mouse_face_mouse_x = x;
+  mouse_face_mouse_y = y;
+  mouse_face_mouse_frame = f;
+
+  if (mouse_face_defer)
+    return;
+
+  if (gc_in_progress)
+    {
+      mouse_face_deferred_gc = 1;
+      return;
+    }
+
+  /* Find out which glyph the mouse is on.  */
+  pixel_to_glyph_coords (f, x, y, &column, &row,
+                        &new_glyph, x_mouse_grabbed);
+
+  /* Which window is that in?  */
+  window = window_from_coordinates (f, column, row, &portion);
+  w = XWINDOW (window);
+
+  /* If we were displaying active text in another window, clear that.  */
+  if (! EQ (window, mouse_face_window))
+    clear_mouse_face ();
+
+  /* Are we in a window whose display is up to date?
+     And verify the buffer's text has not changed.  */
+  if (WINDOWP (window) && portion == 0 && row >= 0 && column >= 0
+      && row < FRAME_HEIGHT (f) && column < FRAME_WIDTH (f)
+      && EQ (w->window_end_valid, w->buffer)
+      && w->last_modified == BUF_MODIFF (XBUFFER (w->buffer)))
+    {
+      int *ptr = FRAME_CURRENT_GLYPHS (f)->charstarts[row];
+      int i, pos;
+
+      /* Find which buffer position the mouse corresponds to.  */
+      for (i = column; i >= 0; i--)
+       if (ptr[i] > 0)
+         break;
+      pos = ptr[i];
+      /* Is it outside the displayed active region (if any)?  */
+      if (pos <= 0)
+       clear_mouse_face ();
+      else if (! (EQ (window, mouse_face_window)
+                 && row >= mouse_face_beg_row
+                 && row <= mouse_face_end_row
+                 && (row > mouse_face_beg_row || column >= mouse_face_beg_col)
+                 && (row < mouse_face_end_row || column < mouse_face_end_col
+                     || mouse_face_past_end)))
+       {
+         Lisp_Object mouse_face, overlay, position;
+         Lisp_Object *overlay_vec;
+         int len, noverlays, ignor1;
+         struct buffer *obuf;
+         int obegv, ozv;
+
+         /* If we get an out-of-range value, return now; avoid an error.  */
+         if (pos > BUF_Z (XBUFFER (w->buffer)))
+           return;
+
+         /* Make the window's buffer temporarily current for
+            overlays_at and compute_char_face.  */
+         obuf = current_buffer;
+         current_buffer = XBUFFER (w->buffer);
+         obegv = BEGV;
+         ozv = ZV;
+         BEGV = BEG;
+         ZV = Z;
+
+         /* Yes.  Clear the display of the old active region, if any.  */
+         clear_mouse_face ();
+
+         /* Is this char mouse-active?  */
+         XSET (position, Lisp_Int, pos);
+
+         len = 10;
+         overlay_vec = (Lisp_Object *) xmalloc (len * sizeof (Lisp_Object));
+
+         /* Put all the overlays we want in a vector in overlay_vec.
+            Store the length in len.  */
+         noverlays = overlays_at (XINT (pos), 1, &overlay_vec, &len, &ignor1);
+         noverlays = sort_overlays (overlay_vec, noverlays, w);
+
+         /* Find the highest priority overlay that has a mouse-face prop.  */
+         overlay = Qnil;
+         for (i = 0; i < noverlays; i++)
+           {
+             mouse_face = Foverlay_get (overlay_vec[i], Qmouse_face);
+             if (!NILP (mouse_face))
+               {
+                 overlay = overlay_vec[i];
+                 break;
+               }
+           }
+         free (overlay_vec);
+         /* If no overlay applies, get a text property.  */
+         if (NILP (overlay))
+           mouse_face = Fget_text_property (position, Qmouse_face, w->buffer);
+
+         /* Handle the overlay case.  */
+         if (! NILP (overlay))
+           {
+             /* Find the range of text around this char that
+                should be active.  */
+             Lisp_Object before, after;
+             int ignore;
+
+             before = Foverlay_start (overlay);
+             after = Foverlay_end (overlay);
+             /* Record this as the current active region.  */
+             fast_find_position (window, before, &mouse_face_beg_col,
+                                 &mouse_face_beg_row);
+             mouse_face_past_end
+               = !fast_find_position (window, after, &mouse_face_end_col,
+                                      &mouse_face_end_row);
+             mouse_face_window = window;
+             mouse_face_face_id = compute_char_face (f, w, pos, 0, 0,
+                                                     &ignore, pos + 1, 1);
+
+             /* Display it as active.  */
+             show_mouse_face (1);
+           }
+         /* Handle the text property case.  */
+         else if (! NILP (mouse_face))
+           {
+             /* Find the range of text around this char that
+                should be active.  */
+             Lisp_Object before, after, beginning, end;
+             int ignore;
+
+             beginning = Fmarker_position (w->start);
+             XSET (end, Lisp_Int,
+                   (BUF_Z (XBUFFER (w->buffer))
+                    - XFASTINT (w->window_end_pos)));
+             before
+               = Fprevious_single_property_change (make_number (pos + 1),
+                                                   Qmouse_face,
+                                                   w->buffer, beginning);
+             after
+               = Fnext_single_property_change (position, Qmouse_face,
+                                               w->buffer, end);
+             /* Record this as the current active region.  */
+             fast_find_position (window, before, &mouse_face_beg_col,
+                                 &mouse_face_beg_row);
+             mouse_face_past_end
+               = !fast_find_position (window, after, &mouse_face_end_col,
+                                      &mouse_face_end_row);
+             mouse_face_window = window;
+             mouse_face_face_id
+               = compute_char_face (f, w, pos, 0, 0,
+                                    &ignore, pos + 1, 1);
+
+             /* Display it as active.  */
+             show_mouse_face (1);
+           }
+         BEGV = obegv;
+         ZV = ozv;
+         current_buffer = obuf;
+       }
+    }
+}
+\f
+/* Find the row and column of position POS in window WINDOW.
+   Store them in *COLUMNP and *ROWP.
+   This assumes display in WINDOW is up to date.
+   If POS is above start of WINDOW, return coords
+   of start of first screen line.
+   If POS is after end of WINDOW, return coords of end of last screen line.
+
+   Value is 1 if POS is in range, 0 if it was off screen.  */
+
+static int
+fast_find_position (window, pos, columnp, rowp)
+     Lisp_Object window;
+     int pos;
+     int *columnp, *rowp;
+{
+  struct window *w = XWINDOW (window);
+  FRAME_PTR f = XFRAME (WINDOW_FRAME (w));
+  int i;
+  int row = 0;
+  int left = w->left;
+  int top = w->top;
+  int height = XFASTINT (w->height) - ! MINI_WINDOW_P (w);
+  int width = window_internal_width (w);
+  int *charstarts;
+  int lastcol;
+
+  /* Find the right row.  */
+  for (i = 0;
+       i < height;
+       i++)
+    {
+      int linestart = FRAME_CURRENT_GLYPHS (f)->charstarts[top + i][left];
+      if (linestart > pos)
+       break;
+      if (linestart > 0)
+       row = i;
+    }
+
+  /* Find the right column with in it.  */
+  charstarts = FRAME_CURRENT_GLYPHS (f)->charstarts[top + row];
+  lastcol = left;
+  for (i = 0; i < width; i++)
+    {
+      if (charstarts[left + i] == pos)
+       {
+         *rowp = row + top;
+         *columnp = i + left;
+         return 1;
+       }
+      else if (charstarts[left + i] > pos)
+       break;
+      else if (charstarts[left + i] > 0)
+       lastcol = left + i;
+    }
+
+  *rowp = row + top;
+  *columnp = lastcol;
+  return 0;
+}
+
+/* Display the active region described by mouse_face_*
+   in its mouse-face if HL > 0, in its normal face if HL = 0.  */
+
+static void
+show_mouse_face (hl)
+     int hl;
+{
+  struct window *w = XWINDOW (mouse_face_window);
+  int width = window_internal_width (w);
+  FRAME_PTR f = XFRAME (WINDOW_FRAME (w));
+  int i;
+  int cursor_off = 0;
+  int old_curs_x = curs_x;
+  int old_curs_y = curs_y;
+
+  /* Set these variables temporarily
+     so that if we have to turn the cursor off and on again
+     we will put it back at the same place.  */
+  curs_x = f->phys_cursor_x;
+  curs_y = f->phys_cursor_y;
+
+  for (i = mouse_face_beg_row; i <= mouse_face_end_row; i++)
+    {
+      int column = (i == mouse_face_beg_row ? mouse_face_beg_col : w->left);
+      int endcolumn = (i == mouse_face_end_row ? mouse_face_end_col : w->left + width);
+      endcolumn = min (endcolumn, FRAME_CURRENT_GLYPHS (f)->used[i]);
+
+      /* If the cursor's in the text we are about to rewrite,
+        turn the cursor off.  */
+      if (i == curs_y
+         && curs_x >= mouse_face_beg_col - 1 && curs_x <= mouse_face_end_col)
+       {
+         x_display_cursor (f, 0);
+         cursor_off = 1;
+       }
+
+      dumpglyphs (f,
+                 CHAR_TO_PIXEL_COL (f, column),
+                 CHAR_TO_PIXEL_ROW (f, i),
+                 FRAME_CURRENT_GLYPHS (f)->glyphs[i] + column,
+                 endcolumn - column,
+                 /* Highlight with mouse face if hl > 0.  */
+                 hl > 0 ? 3 : 0, 0);
+    }
+
+  /* If we turned the cursor off, turn it back on.  */
+  if (cursor_off)
+    x_display_cursor (f, 1);
+
+  curs_x = old_curs_x;
+  curs_y = old_curs_y;
+
+  /* Change the mouse cursor according to the value of HL.  */
+  if (hl > 0)
+    XDefineCursor (XDISPLAY FRAME_X_WINDOW (f), f->display.x->cross_cursor);
+  else
+    XDefineCursor (XDISPLAY FRAME_X_WINDOW (f), f->display.x->text_cursor);
+}
+
+/* Clear out the mouse-highlighted active region.
+   Redraw it unhighlighted first.  */
+
+static void
+clear_mouse_face ()
+{
+  if (! NILP (mouse_face_window))
+    show_mouse_face (0);
+
+  mouse_face_beg_row = mouse_face_beg_col = -1;
+  mouse_face_end_row = mouse_face_end_col = -1;
+  mouse_face_window = Qnil;
+}
+\f
 static struct scroll_bar *x_window_to_scroll_bar ();
 static void x_scroll_bar_report_motion ();
 
@@ -1695,6 +2452,8 @@ static void x_scroll_bar_report_motion ();
    Set *time to the server timestamp for the time at which the mouse
    was at this position.
 
+   Don't store anything if we don't have a valid set of values to report.
+
    This clears the mouse_moved flag, so we can wait for the next mouse
    movement.  This also calls XQueryPointer, which will cause the
    server to give us another MotionNotify when the mouse moves
@@ -1708,6 +2467,8 @@ XTmouse_position (f, bar_window, part, x, y, time)
      Lisp_Object *x, *y;
      unsigned long *time;
 {
+  FRAME_PTR f1;
+
   BLOCK_INPUT;
 
   if (! NILP (last_mouse_scroll_bar))
@@ -1752,61 +2513,87 @@ XTmouse_position (f, bar_window, part, x, y, time)
        int parent_x, parent_y;
 
        win = root;
-       for (;;)
+
+       if (x_mouse_grabbed && last_mouse_frame
+           && FRAME_LIVE_P (last_mouse_frame))
          {
+           /* If mouse was grabbed on a frame, give coords for that frame
+              even if the mouse is now outside it.  */
            XTranslateCoordinates (x_current_display,
-                              
+
                                   /* From-window, to-window.  */
-                                  root, win,
+                                  root, FRAME_X_WINDOW (last_mouse_frame),
 
                                   /* From-position, to-position.  */
                                   root_x, root_y, &win_x, &win_y,
 
                                   /* Child of win.  */
                                   &child);
+           f1 = last_mouse_frame;
+         }
+       else
+         {
+           while (1)
+             {
+               XTranslateCoordinates (x_current_display,
 
-           if (child == None)
-             break;
+                                      /* From-window, to-window.  */
+                                      root, win,
 
-           win = child;
-           parent_x = win_x;
-           parent_y = win_y;
-         }
+                                      /* From-position, to-position.  */
+                                      root_x, root_y, &win_x, &win_y,
+
+                                      /* Child of win.  */
+                                      &child);
 
-       /* Now we know that:
-          win is the innermost window containing the pointer
-          (XTC says it has no child containing the pointer),
-          win_x and win_y are the pointer's position in it
-          (XTC did this the last time through), and
-          parent_x and parent_y are the pointer's position in win's parent.
-          (They are what win_x and win_y were when win was child.
-          If win is the root window, it has no parent, and
-          parent_{x,y} are invalid, but that's okay, because we'll
-          never use them in that case.)  */
-
-       /* Is win one of our frames?  */
-       *f = x_window_to_frame (win);
+               if (child == None)
+                 break;
+
+               win = child;
+               parent_x = win_x;
+               parent_y = win_y;
+             }
+
+           /* Now we know that:
+              win is the innermost window containing the pointer
+              (XTC says it has no child containing the pointer),
+              win_x and win_y are the pointer's position in it
+              (XTC did this the last time through), and
+              parent_x and parent_y are the pointer's position in win's parent.
+              (They are what win_x and win_y were when win was child.
+              If win is the root window, it has no parent, and
+              parent_{x,y} are invalid, but that's okay, because we'll
+              never use them in that case.)  */
+
+           /* Is win one of our frames?  */
+           f1 = x_any_window_to_frame (win);
+         }
       
        /* If not, is it one of our scroll bars?  */
-       if (! *f)
+       if (! f1)
          {
            struct scroll_bar *bar = x_window_to_scroll_bar (win);
 
            if (bar)
              {
-               *f = XFRAME (WINDOW_FRAME (XWINDOW (bar->window)));
+               f1 = XFRAME (WINDOW_FRAME (XWINDOW (bar->window)));
                win_x = parent_x;
                win_y = parent_y;
              }
          }
 
-       if (*f)
+       if (f1)
          {
-           pixel_to_glyph_coords (*f, win_x, win_y, &win_x, &win_y,
-                                  &last_mouse_glyph);
+           int ignore1, ignore2;
+
+           /* Ok, we found a frame.  Store all the values.  */
+
+           pixel_to_glyph_coords (f1, win_x, win_y, &ignore1, &ignore2,
+                                  &last_mouse_glyph, x_mouse_grabbed);
 
            *bar_window = Qnil;
            *part = 0;
+           *f = f1;
            XSET (*x, Lisp_Int, win_x);
            XSET (*y, Lisp_Int, win_y);
            *time = last_mouse_movement_time;
@@ -1836,9 +2623,9 @@ x_window_to_scroll_bar (window_id)
        XGCTYPE (tail) == Lisp_Cons;
        tail = XCONS (tail)->cdr)
     {
-      Lisp_Object frame = XCONS (tail)->car;
-      Lisp_Object bar, condemned;
+      Lisp_Object frame, bar, condemned;
 
+      frame = XCONS (tail)->car;
       /* All elements of Vframe_list should be frames.  */
       if (XGCTYPE (frame) != Lisp_Frame)
        abort ();
@@ -1852,7 +2639,7 @@ x_window_to_scroll_bar (window_id)
           ! GC_NILP (bar) || (bar = condemned,
                               condemned = Qnil,
                               ! GC_NILP (bar));
-          bar = XSCROLL_BAR(bar)->next)
+          bar = XSCROLL_BAR (bar)->next)
        if (SCROLL_BAR_X_WINDOW (XSCROLL_BAR (bar)) == window_id)
          return XSCROLL_BAR (bar);
     }
@@ -1876,7 +2663,6 @@ x_scroll_bar_create (window, top, left, width, height)
   {
     XSetWindowAttributes a;
     unsigned long mask;
-
     a.background_pixel = frame->display.x->background_pixel;
     a.event_mask = (ButtonPressMask | ButtonReleaseMask
                    | ButtonMotionMask | PointerMotionHintMask
@@ -1885,6 +2671,20 @@ x_scroll_bar_create (window, top, left, width, height)
 
     mask = (CWBackPixel | CWEventMask | CWCursor);
 
+#if 0
+
+    ac = 0;
+    XtSetArg (al[ac], XtNx, left); ac++;
+    XtSetArg (al[ac], XtNy, top); ac++;
+    XtSetArg (al[ac], XtNwidth, width); ac++;
+    XtSetArg (al[ac], XtNheight, height); ac++;
+    XtSetArg (al[ac], XtNborderWidth, 0); ac++;
+    sb_widget = XtCreateManagedWidget ("box",
+                                      boxWidgetClass,
+                                      frame->display.x->edit_widget, al, ac);
+    SET_SCROLL_BAR_X_WINDOW
+      (bar, sb_widget->core.window);
+#endif    
     SET_SCROLL_BAR_X_WINDOW
       (bar, 
        XCreateWindow (x_current_display, FRAME_X_WINDOW (frame),
@@ -2022,7 +2822,7 @@ x_scroll_bar_set_handle (bar, start, end, rebuild)
   UNBLOCK_INPUT;
 }
 
-/* Move a scroll bar around on the screen, to accomodate changing
+/* Move a scroll bar around on the screen, to accommodate changing
    window configurations.  */
 static void
 x_scroll_bar_move (bar, top, left, width, height)
@@ -2122,8 +2922,8 @@ XTset_vertical_scroll_bar (window, portion, whole, position)
        x_scroll_bar_set_handle (bar, 0, top_range, 0);
       else
        {
-         int start = (position * top_range) / whole;
-         int end = ((position + portion) * top_range) / whole;
+         int start = ((double) position * top_range) / whole;
+         int end = ((double) (position + portion) * top_range) / whole;
 
          x_scroll_bar_set_handle (bar, start, end, 0);
        }
@@ -2257,12 +3057,6 @@ x_scroll_bar_expose (bar, event)
                  /* x, y, width, height */
                  0, 0, XINT (bar->width) - 1, XINT (bar->height) - 1);
 
-  /* Draw another line to make the extra-thick border on the right.  */
-  XFillRectangle (x_current_display, w, gc,
-
-                 /* x, y, width, height */
-                 XINT (bar->width) - 2, 1, 1, XINT (bar->height) - 2);
-
   UNBLOCK_INPUT;
 }
 
@@ -2281,7 +3075,7 @@ x_scroll_bar_handle_click (bar, event, emacs_event)
     abort ();
 
   emacs_event->kind = scroll_bar_click;
-  XSET (emacs_event->code, Lisp_Int, event->xbutton.button - Button1);
+  emacs_event->code = event->xbutton.button - Button1;
   emacs_event->modifiers =
     (x_x_to_emacs_modifiers (event->xbutton.state)
      | (event->type == ButtonRelease
@@ -2330,7 +3124,7 @@ x_scroll_bar_handle_click (bar, event, emacs_event)
 
     /* Same deal here as the other #if 0.  */
 #if 0
-    /* Clicks on the handle are always reported as occuring at the top of 
+    /* Clicks on the handle are always reported as occurring at the top of 
        the handle.  */
     if (emacs_event->part == scroll_bar_handle)
       emacs_event->x = bar->start;
@@ -2376,9 +3170,10 @@ x_scroll_bar_note_movement (bar, event)
      moves and we can see *still* on the same position.  */
   {
     int dummy;
+    Window dummy_window;
       
     XQueryPointer (event->xmotion.display, event->xmotion.window,
-                  (Window *) &dummy, (Window *) &dummy,
+                  &dummy_window, &dummy_window,
                   &dummy, &dummy, &dummy, &dummy,
                   (unsigned int *) &dummy);
   }
@@ -2476,7 +3271,107 @@ x_scroll_bar_clear (f)
                0, 0, 0, 0, True);
 }
 
+/* This processes Expose events from the menubar specific X event
+   loop in menubar.c.  This allows to redisplay the frame if necessary
+   when handling menubar or popup items.  */
+
+void
+process_expose_from_menu (event)
+     XEvent event;
+{
+  FRAME_PTR f;
+
+  BLOCK_INPUT;
+
+  f = x_window_to_frame (event.xexpose.window);
+  if (f)
+    {
+      if (f->async_visible == 0)
+       {
+         f->async_visible = 1;
+         f->async_iconified = 0;
+         SET_FRAME_GARBAGED (f);
+       }
+      else
+       {
+         dumprectangle (x_window_to_frame (event.xexpose.window),
+                        event.xexpose.x, event.xexpose.y,
+                        event.xexpose.width, event.xexpose.height);
+       }
+    }
+  else
+    {
+      struct scroll_bar *bar
+       = x_window_to_scroll_bar (event.xexpose.window);
+      
+      if (bar)
+       x_scroll_bar_expose (bar, &event);
+    }
+
+  UNBLOCK_INPUT;
+}
+\f
+/* Define a queue to save up SelectionRequest events for later handling.  */
+
+struct selection_event_queue
+  {
+    XEvent event;
+    struct selection_event_queue *next;
+  };
+
+static struct selection_event_queue *queue;
+
+/* Nonzero means queue up certain events--don't process them yet.  */
+static int x_queue_selection_requests;
+
+/* Queue up an X event *EVENT, to be processed later.  */
+
+static void
+x_queue_event (event)
+     XEvent *event;
+{
+  struct selection_event_queue *queue_tmp
+    = (struct selection_event_queue *) malloc (sizeof (struct selection_event_queue));
+
+  if (queue_tmp != NULL) 
+    {
+      queue_tmp->event = *event;
+      queue_tmp->next = queue;
+      queue = queue_tmp;
+    }
+}
+
+/* Take all the queued events and put them back
+   so that they get processed afresh.  */
+
+static void
+x_unqueue_events ()
+{
+  while (queue != NULL) 
+    {
+      struct selection_event_queue *queue_tmp = queue;
+      XPutBackEvent (XDISPLAY &queue_tmp->event);
+      queue = queue_tmp->next;
+      free ((char *)queue_tmp);
+    }
+}
+
+/* Start queuing SelectionRequest events.  */
 
+void
+x_start_queuing_selection_requests ()
+{
+  x_queue_selection_requests++;
+}
+
+/* Stop queuing SelectionRequest events.  */
+
+void
+x_stop_queuing_selection_requests ()
+{
+  x_queue_selection_requests--;
+  x_unqueue_events ();
+}
 \f
 /* The main X event-reading loop - XTread_socket.  */
 
@@ -2509,6 +3404,9 @@ Atom Xatom_wm_window_moved;         /* When the WM moves us. */
 /* Window manager communication.  */
 Atom Xatom_wm_change_state;
 
+/* EditRes protocol */
+Atom Xatom_editres_name;
+
 /* Record the last 100 characters stored
    to help debug the loss-of-chars-during-GC problem.  */
 int temp_index;
@@ -2540,7 +3438,7 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
   int items_pending;           /* How many items are in the X queue. */
   XEvent event;
   struct frame *f;
-  int event_found;
+  int event_found = 0;
   int prefix;
   Lisp_Object part;
 
@@ -2592,8 +3490,14 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                if (event.xclient.data.l[0] == Xatom_wm_take_focus)
                  {
                    f = x_window_to_frame (event.xclient.window);
+                   /* Since we set WM_TAKE_FOCUS, we must call
+                      XSetInputFocus explicitly.  But not if f is null,
+                      since that might be an event for a deleted frame.  */
                    if (f)
-                     x_focus_on_frame (f);
+                     XSetInputFocus (event.xclient.display,
+                                     event.xclient.window,
+                                     RevertToPointerRoot,
+                                     event.xclient.data.l[1]);
                    /* Not certain about handling scroll bars here */
                  }
                else if (event.xclient.data.l[0] == Xatom_wm_save_yourself)
@@ -2605,16 +3509,35 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                       a keyboard or mouse event arrives. */
                    if (numchars > 0)
                      {
+                       /* This is just so we only give real data once
+                          for a single Emacs process.  */
+                       if (x_top_window_to_frame (event.xclient.window)
+                           == selected_frame)
+                         XSetCommand (x_current_display,
+                                      event.xclient.window,
+                                      initial_argv, initial_argc);
+                       else
+                         XSetCommand (x_current_display,
+                                      event.xclient.window,
+                                      0, 0);
                      }
                  }
                else if (event.xclient.data.l[0] == Xatom_wm_delete_window)
                  {
-                   struct frame *f = x_window_to_frame (event.xclient.window);
+                   struct frame *f = x_any_window_to_frame (event.xclient.window);
 
                    if (f)
-                     if (numchars > 0)
-                       {
-                       }
+                     {
+                       if (numchars == 0)
+                         abort ();
+
+                       bufp->kind = delete_window_event;
+                       XSET (bufp->frame_or_window, Lisp_Frame, f);
+                       bufp++;
+
+                       count += 1;
+                       numchars -= 1;
+                     }
                  }
              }
            else if (event.xclient.message_type == Xatom_wm_configure_denied)
@@ -2623,21 +3546,40 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
            else if (event.xclient.message_type == Xatom_wm_window_moved)
              {
                int new_x, new_y;
-
+               struct frame *f = x_window_to_frame (event.xclient.window);
+               
                new_x = event.xclient.data.s[0];
                new_y = event.xclient.data.s[1];
+
+               if (f)
+                 {
+                   f->display.x->left_pos = new_x;
+                   f->display.x->top_pos = new_y;
+                 }
+             }
+#if defined (USE_X_TOOLKIT) && defined (HAVE_X11R5)
+           else if (event.xclient.message_type == Xatom_editres_name)
+             {
+               struct frame *f = x_any_window_to_frame (event.xclient.window);
+               _XEditResCheckMessages (f->display.x->widget, NULL, &event, NULL);
              }
+#endif /* USE_X_TOOLKIT and HAVE_X11R5 */
          }
          break;
 
-#ifdef NEW_SELECTIONS
        case SelectionNotify:
+#ifdef USE_X_TOOLKIT
+         if (! x_window_to_frame (event.xselection.requestor))
+           goto OTHER;
+#endif /* not USE_X_TOOLKIT */
          x_handle_selection_notify (&event);
          break;
-#endif
 
        case SelectionClear:    /* Someone has grabbed ownership. */
-#ifdef NEW_SELECTIONS
+#ifdef USE_X_TOOLKIT
+         if (! x_window_to_frame (event.xselectionclear.window))
+           goto OTHER;
+#endif /* USE_X_TOOLKIT */
          {
            XSelectionClearEvent *eventp = (XSelectionClearEvent *) &event;
 
@@ -2653,62 +3595,54 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
            count += 1;
            numchars -= 1;
          }
-#else
-         x_disown_selection (event.xselectionclear.window,
-                             event.xselectionclear.selection,
-                             event.xselectionclear.time);
-#endif
          break;
 
        case SelectionRequest:  /* Someone wants our selection. */
-#ifdef NEW_SELECTIONS
-         {
-           XSelectionRequestEvent *eventp = (XSelectionRequestEvent *) &event;
-
-           if (numchars == 0)
-             abort ();
-
-           bufp->kind = selection_request_event;
-           SELECTION_EVENT_DISPLAY (bufp) = eventp->display;
-           SELECTION_EVENT_REQUESTOR (bufp) = eventp->requestor;
-           SELECTION_EVENT_SELECTION (bufp) = eventp->selection;
-           SELECTION_EVENT_TARGET (bufp) = eventp->target;
-           SELECTION_EVENT_PROPERTY (bufp) = eventp->property;
-           SELECTION_EVENT_TIME (bufp) = eventp->time;
-           bufp++;
+#ifdef USE_X_TOOLKIT
+         if (!x_window_to_frame (event.xselectionrequest.owner))
+           goto OTHER;
+#endif /* USE_X_TOOLKIT */
+         if (x_queue_selection_requests)
+           x_queue_event (&event);
+         else
+           {
+             XSelectionRequestEvent *eventp = (XSelectionRequestEvent *) &event;
 
-           count += 1;
-           numchars -= 1;
-         }
-#else
-         x_answer_selection_request (event);
-#endif
-         break;
+             if (numchars == 0)
+               abort ();
 
-       case PropertyNotify:
-#ifdef NEW_SELECTIONS
-         x_handle_property_notify (&event);
-#else
-         /* If we're being told about a root window property, then it's
-            a cut buffer change.  */
-         if (event.xproperty.window == ROOT_WINDOW)
-           x_invalidate_cut_buffer_cache (&event.xproperty);
+             bufp->kind = selection_request_event;
+             SELECTION_EVENT_DISPLAY (bufp) = eventp->display;
+             SELECTION_EVENT_REQUESTOR (bufp) = eventp->requestor;
+             SELECTION_EVENT_SELECTION (bufp) = eventp->selection;
+             SELECTION_EVENT_TARGET (bufp) = eventp->target;
+             SELECTION_EVENT_PROPERTY (bufp) = eventp->property;
+             SELECTION_EVENT_TIME (bufp) = eventp->time;
+             bufp++;
 
-         /* Otherwise, we're probably handling an incremental
-             selection transmission.  */
-         else
-           {
-             /* If we were to do this synchronously, there'd be no worry
-                about re-selecting. */
-             x_send_incremental (event);
+             count += 1;
+             numchars -= 1;
            }
-#endif
+         break;
+
+       case PropertyNotify:
+#ifdef USE_X_TOOLKIT
+         if (!x_any_window_to_frame (event.xproperty.window))
+           goto OTHER;
+#endif /* not USE_X_TOOLKIT */
+         x_handle_property_notify (&event);
          break;
 
        case ReparentNotify:
-         f = x_window_to_frame (event.xreparent.window);
+         f = x_top_window_to_frame (event.xreparent.window);
          if (f)
-           f->display.x->parent_desc = event.xreparent.parent;
+           {
+             int x, y;
+             f->display.x->parent_desc = event.xreparent.parent;
+             x_real_positions (f, &x, &y);
+             f->display.x->left_pos = x;
+             f->display.x->top_pos = y;
+           }
          break;
 
        case Expose:
@@ -2722,19 +3656,21 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                  SET_FRAME_GARBAGED (f);
                }
              else
-               {
-                 dumprectangle (x_window_to_frame (event.xexpose.window),
-                                event.xexpose.x, event.xexpose.y,
-                                event.xexpose.width, event.xexpose.height);
-               }
+               dumprectangle (x_window_to_frame (event.xexpose.window),
+                              event.xexpose.x, event.xexpose.y,
+                              event.xexpose.width, event.xexpose.height);
            }
          else
            {
              struct scroll_bar *bar
                = x_window_to_scroll_bar (event.xexpose.window);
-
+             
              if (bar)
-               x_scroll_bar_expose (bar, &event);
+               x_scroll_bar_expose (bar, &event);
+#ifdef USE_X_TOOLKIT
+             else
+               goto OTHER;
+#endif /* USE_X_TOOLKIT */
            }
          break;
 
@@ -2749,6 +3685,10 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                             event.xgraphicsexpose.width,
                             event.xgraphicsexpose.height);
            }
+#ifdef USE_X_TOOLKIT
+         else
+           goto OTHER;
+#endif /* USE_X_TOOLKIT */
          break;
 
        case NoExpose:          /* This occurs when an XCopyArea's
@@ -2805,7 +3745,7 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
 
 #ifdef HAVE_X11
        case UnmapNotify:
-         f = x_window_to_frame (event.xunmap.window);
+         f = x_any_window_to_frame (event.xunmap.window);
          if (f)                /* F may no longer exist if
                                   the frame was deleted.  */
            {
@@ -2813,17 +3753,22 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                 disabled; you don't want to spend time updating a
                 display that won't ever be seen.  */
              f->async_visible = 0;
-             /* The window manager never makes a window invisible
-                ("withdrawn"); all it does is switch between visible
-                and iconified.  Frames get into the invisible state
-                only through x_make_frame_invisible.
+             /* We can't distinguish, from the event, whether the window
+                has become iconified or invisible.  So assume, if it
+                was previously visible, than now it is iconified.
+                We depend on x_make_frame_invisible to mark it iconified.  */
              if (FRAME_VISIBLE_P (f) || FRAME_ICONIFIED_P (f))
                f->async_iconified = 1;
            }
+#ifdef USE_X_TOOLKIT
+         goto OTHER;
+#endif /* USE_X_TOOLKIT */
          break;
 
        case MapNotify:
-         f = x_window_to_frame (event.xmap.window);
+         /* We use x_top_window_to_frame because map events can come
+            for subwindows and they don't mean that the frame is visible.  */
+         f = x_top_window_to_frame (event.xmap.window);
          if (f)
            {
              f->async_visible = 1;
@@ -2833,6 +3778,9 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                 the frame's display structures.  */
              SET_FRAME_GARBAGED (f);
            }
+#ifdef USE_X_TOOLKIT
+         goto OTHER;
+#endif /* USE_X_TOOLKIT */
          break;
 
          /* Turn off processing if we become fully obscured. */
@@ -2851,12 +3799,14 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
 
 #ifdef HAVE_X11
        case KeyPress:
-         f = x_window_to_frame (event.xkey.window);
+         f = x_any_window_to_frame (event.xkey.window);
 
          if (f != 0)
            {
-             KeySym keysym;
-             char copy_buffer[80];
+             KeySym keysym, orig_keysym;
+             /* al%imercury@uunet.uu.net says that making this 81 instead of
+                80 fixed a bug whereby meta chars made his Emacs hang.  */
+             unsigned char copy_buffer[81];
              int modifiers;
 
              event.xkey.state
@@ -2872,31 +3822,81 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                XLookupString (&event.xkey, copy_buffer, 80, &keysym,
                               &compose_status);
 
-             /* Strip off the vendor-specific keysym bit, and take a shot
-                at recognizing the codes.  HP servers have extra keysyms
-                that fit into the MiscFunctionKey category.  */
-             keysym &= ~(1<<28);
+             orig_keysym = keysym;
 
              if (numchars > 1)
                {
-                 if ((keysym >= XK_BackSpace && keysym <= XK_Escape)
-                     || keysym == XK_Delete
-                     || IsCursorKey (keysym)       /* 0xff50 <= x < 0xff60 */
-                     || IsMiscFunctionKey (keysym) /* 0xff60 <= x < 0xff80 */
+                 if (((keysym >= XK_BackSpace && keysym <= XK_Escape)
+                      || keysym == XK_Delete
+                      || IsCursorKey (keysym) /* 0xff50 <= x < 0xff60 */
+                      || IsMiscFunctionKey (keysym) /* 0xff60 <= x < VARIES */
 #ifdef HPUX
-                     /* This recognizes the "extended function keys".
-                        It seems there's no cleaner way.  */
-                     || ((unsigned) (keysym) >= XK_Select
-                         && (unsigned)(keysym) < XK_KP_Space)
+                      /* This recognizes the "extended function keys".
+                         It seems there's no cleaner way.
+                         Test IsModifierKey to avoid handling mode_switch
+                         incorrectly.  */
+                      || ((unsigned) (keysym) >= XK_Select
+                          && (unsigned)(keysym) < XK_KP_Space)
+#endif
+#ifdef XK_dead_circumflex
+                      || orig_keysym == XK_dead_circumflex
+#endif
+#ifdef XK_dead_grave
+                      || orig_keysym == XK_dead_grave
+#endif
+#ifdef XK_dead_tilde
+                      || orig_keysym == XK_dead_tilde
 #endif
-                     || IsKeypadKey (keysym)       /* 0xff80 <= x < 0xffbe */
-                     || IsFunctionKey (keysym))    /* 0xffbe <= x < 0xffe1 */
+#ifdef XK_dead_diaeresis
+                      || orig_keysym == XK_dead_diaeresis
+#endif
+#ifdef XK_dead_macron
+                      || orig_keysym == XK_dead_macron
+#endif
+#ifdef XK_dead_degree
+                      || orig_keysym == XK_dead_degree
+#endif
+#ifdef XK_dead_acute
+                      || orig_keysym == XK_dead_acute
+#endif
+#ifdef XK_dead_cedilla
+                      || orig_keysym == XK_dead_cedilla
+#endif
+#ifdef XK_dead_breve
+                      || orig_keysym == XK_dead_breve
+#endif
+#ifdef XK_dead_ogonek
+                      || orig_keysym == XK_dead_ogonek
+#endif
+#ifdef XK_dead_caron
+                      || orig_keysym == XK_dead_caron
+#endif
+#ifdef XK_dead_doubleacute
+                      || orig_keysym == XK_dead_doubleacute
+#endif
+#ifdef XK_dead_abovedot
+                      || orig_keysym == XK_dead_abovedot
+#endif
+                      || IsKeypadKey (keysym) /* 0xff80 <= x < 0xffbe */
+                      || IsFunctionKey (keysym) /* 0xffbe <= x < 0xffe1 */
+                      /* Any "vendor-specific" key is ok.  */
+                      || (orig_keysym & (1 << 28)))
+                     && ! (IsModifierKey (orig_keysym)
+#ifndef HAVE_X11R5
+#ifdef XK_Mode_switch
+                           || ((unsigned)(orig_keysym) == XK_Mode_switch)
+#endif
+#ifdef XK_Num_Lock
+                           || ((unsigned)(orig_keysym) == XK_Num_Lock)
+#endif
+#endif /* not HAVE_X11R5 */
+                           ))
                    {
                      if (temp_index == sizeof temp_buffer / sizeof (short))
                        temp_index = 0;
                      temp_buffer[temp_index++] = keysym;
                      bufp->kind = non_ascii_keystroke;
-                     XSET (bufp->code, Lisp_Int, (unsigned) keysym - 0xff00);
+                     bufp->code = keysym;
                      XSET (bufp->frame_or_window, Lisp_Frame, f);
                      bufp->modifiers = x_x_to_emacs_modifiers (modifiers);
                      bufp->timestamp = event.xkey.time;
@@ -2914,7 +3914,7 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                            temp_index = 0;
                          temp_buffer[temp_index++] = copy_buffer[i];
                          bufp->kind = ascii_keystroke;
-                         XSET (bufp->code, Lisp_Int, copy_buffer[i]);
+                         bufp->code = copy_buffer[i];
                          XSET (bufp->frame_or_window, Lisp_Frame, f);
                          bufp->modifiers = x_x_to_emacs_modifiers (modifiers);
                          bufp->timestamp = event.xkey.time;
@@ -2968,7 +3968,7 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                for (i = 0; i < nbytes; i++)
                  {
                    bufp->kind = ascii_keystroke;
-                   XSET (bufp->code, Lisp_Int, where_mapping[i]);
+                   bufp->code = where_mapping[i];
                    XSET (bufp->time, Lisp_Int, event.xkey.time);
                    XSET (bufp->frame_or_window, Lisp_Frame, f);
                    bufp++;
@@ -2989,7 +3989,7 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
             then a mere LeaveNotify is enough to free you.  */
 
        case EnterNotify:
-         f = x_window_to_frame (event.xcrossing.window);
+         f = x_any_window_to_frame (event.xcrossing.window);
 
          if (event.xcrossing.focus)            /* Entered Window */
            {
@@ -3004,44 +4004,66 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
            }
          else if (f == x_focus_frame)
            x_new_focus_frame (0);
-
+         /* EnterNotify counts as mouse movement,
+            so update things that depend on mouse position.  */
+         if (f)
+           note_mouse_movement (f, &event.xmotion);
+#ifdef USE_X_TOOLKIT
+         goto OTHER;
+#endif /* USE_X_TOOLKIT */
          break;
 
        case FocusIn:
-         f = x_window_to_frame (event.xfocus.window);
+         f = x_any_window_to_frame (event.xfocus.window);
          if (event.xfocus.detail != NotifyPointer) 
            x_focus_event_frame = f;
          if (f)
            x_new_focus_frame (f);
+#ifdef USE_X_TOOLKIT
+         goto OTHER;
+#endif /* USE_X_TOOLKIT */
          break;
 
 
        case LeaveNotify:
-         f = x_window_to_frame (event.xcrossing.window);
-
-         if (event.xcrossing.focus)
-           {
-             if (! x_focus_event_frame)
-               x_new_focus_frame (0);
-             else
-               x_new_focus_frame (f);
-           }
-         else 
+         f = x_top_window_to_frame (event.xcrossing.window);
+         if (f)
            {
-             if (f == x_focus_event_frame)
-               x_focus_event_frame = 0;
-             if (f == x_focus_frame)
-               x_new_focus_frame (0);
+             if (f == mouse_face_mouse_frame)
+               /* If we move outside the frame,
+                  then we're certainly no longer on any text in the frame.  */
+               clear_mouse_face ();
+
+             if (event.xcrossing.focus)
+               {
+                 if (! x_focus_event_frame)
+                   x_new_focus_frame (0);
+                 else
+                   x_new_focus_frame (f);
+               }
+             else 
+               {
+                 if (f == x_focus_event_frame)
+                   x_focus_event_frame = 0;
+                 if (f == x_focus_frame)
+                   x_new_focus_frame (0);
+               }
            }
+#ifdef USE_X_TOOLKIT
+         goto OTHER;
+#endif /* USE_X_TOOLKIT */
          break;
 
        case FocusOut:
-         f = x_window_to_frame (event.xfocus.window);
+         f = x_any_window_to_frame (event.xfocus.window);
          if (event.xfocus.detail != NotifyPointer
              && f == x_focus_event_frame)
            x_focus_event_frame = 0;
          if (f && f == x_focus_frame)
            x_new_focus_frame (0);
+#ifdef USE_X_TOOLKIT
+         goto OTHER;
+#endif /* USE_X_TOOLKIT */
          break;
 
 #else /* ! defined (HAVE_X11) */
@@ -3083,22 +4105,85 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
 #ifdef HAVE_X11
        case MotionNotify:
          {
-           f = x_window_to_frame (event.xmotion.window);
+           if (x_mouse_grabbed && last_mouse_frame
+               && FRAME_LIVE_P (last_mouse_frame))
+             f = last_mouse_frame;
+           else
+             f = x_window_to_frame (event.xmotion.window);
            if (f)
              note_mouse_movement (f, &event.xmotion);
            else
              {
-               struct scroll_bar *bar =
-                 x_window_to_scroll_bar (event.xmotion.window);
+               struct scroll_bar *bar
+                 x_window_to_scroll_bar (event.xmotion.window);
 
                if (bar)
                  x_scroll_bar_note_movement (bar, &event);
+
+               /* If we move outside the frame,
+                  then we're certainly no longer on any text in the frame.  */
+               clear_mouse_face ();
              }
          }
+#if 0 /* This should be unnecessary, since the toolkit has no use
+        for motion events that happen outside of the menu event loop,
+        and it seems to cause the bug that mouse events stop coming
+        after a while.  */
+#ifdef USE_X_TOOLKIT
+         goto OTHER;
+#endif /* USE_X_TOOLKIT */
+#endif
          break;
 
        case ConfigureNotify:
-         f = x_window_to_frame (event.xconfigure.window);
+          f = x_any_window_to_frame (event.xconfigure.window);
+#ifdef USE_X_TOOLKIT
+          if (f
+#if 0
+              && ! event.xconfigure.send_event
+#endif
+              && (event.xconfigure.window == XtWindow (f->display.x->widget)))
+            {
+              Window win, child;
+              int win_x, win_y;
+
+              /* Find the position of the outside upper-left corner of
+                 the window, in the root coordinate system.  Don't
+                 refer to the parent window here; we may be processing
+                 this event after the window manager has changed our
+                 parent, but before we have reached the ReparentNotify.  */
+              XTranslateCoordinates (x_current_display,
+
+                                     /* From-window, to-window.  */
+                                     XtWindow (f->display.x->widget),
+                                     ROOT_WINDOW,
+
+                                     /* From-position, to-position.  */
+                                     -event.xconfigure.border_width,
+                                     -event.xconfigure.border_width,
+                                     &win_x, &win_y,
+
+                                     /* Child of win.  */
+                                     &child);
+              event.xconfigure.x = win_x;
+              event.xconfigure.y = win_y;
+
+              f->display.x->pixel_width = event.xconfigure.width;
+              f->display.x->pixel_height = event.xconfigure.height;
+              f->display.x->left_pos = event.xconfigure.x;
+              f->display.x->top_pos = event.xconfigure.y;
+
+             /* What we have now is the position of Emacs's own window.
+                Convert that to the position of the window manager window.  */
+             {
+               int x, y;
+               x_real_positions (f, &x, &y);
+               f->display.x->left_pos = x;
+               f->display.x->top_pos = y;
+             }
+            }
+          goto OTHER;
+#else /* not USE_X_TOOLKIT */
          if (f)
            {
              int rows = PIXEL_TO_CHAR_HEIGHT (f, event.xconfigure.height);
@@ -3121,17 +4206,20 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                  Window win, child;
                  int win_x, win_y;
 
-                 /* Coords are relative to the parent.
-                    Convert them to root-relative.  */
+                 /* Find the position of the outside upper-left corner of
+                    the window, in the root coordinate system.  Don't
+                    refer to the parent window here; we may be processing
+                    this event after the window manager has changed our
+                    parent, but before we have reached the ReparentNotify.  */
                  XTranslateCoordinates (x_current_display,
                               
                                         /* From-window, to-window.  */
-                                        f->display.x->parent_desc,
+                                        f->display.x->window_desc,
                                         ROOT_WINDOW,
 
                                         /* From-position, to-position.  */
-                                        event.xconfigure.x,
-                                        event.xconfigure.y,
+                                        -event.xconfigure.border_width,
+                                        -event.xconfigure.border_width,
                                         &win_x, &win_y,
 
                                         /* Child of win.  */
@@ -3144,7 +4232,24 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
              f->display.x->pixel_height = event.xconfigure.height;
              f->display.x->left_pos = event.xconfigure.x;
              f->display.x->top_pos = event.xconfigure.y;
+
+             /* What we have now is the position of Emacs's own window.
+                Convert that to the position of the window manager window.  */
+             {
+               int x, y;
+               x_real_positions (f, &x, &y);
+               f->display.x->left_pos = x;
+               f->display.x->top_pos = y;
+               if (y != event.xconfigure.y)
+                 {
+                   /* Since the WM decorations come below top_pos now,
+                      we must put them below top_pos in the future.  */
+                   f->display.x->win_gravity = NorthWestGravity;
+                   x_wm_set_size_hint (f, 0, 0);
+                 }
+             }
            }
+#endif /* not USE_X_TOOLKIT */
          break;
 
        case ButtonPress:
@@ -3155,20 +4260,43 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
            struct input_event emacs_event;
            emacs_event.kind = no_event;
 
+           bzero (&compose_status, sizeof (compose_status));
+
            f = x_window_to_frame (event.xbutton.window);
            if (f)
              {
                if (!x_focus_frame || (f == x_focus_frame))
-                 construct_mouse_click (&emacs_event,
-                                        &event, f);
+                 construct_mouse_click (&emacs_event, &event, f);
              }
            else
              {
-               struct scroll_bar *bar =
-                 x_window_to_scroll_bar (event.xbutton.window);
+               struct scroll_bar *bar
+                 x_window_to_scroll_bar (event.xbutton.window);
 
                if (bar)
                  x_scroll_bar_handle_click (bar, &event, &emacs_event);
+#ifdef USE_X_TOOLKIT
+               else
+                 {
+                   f = x_any_window_to_frame (event.xbutton.window);
+                   if (f && event.type == ButtonPress)
+                     construct_menu_click (&emacs_event,
+                                           &event, f);
+                 }
+#endif /* USE_X_TOOLKIT */
+             }
+
+           if (event.type == ButtonPress)
+             {
+               x_mouse_grabbed |= (1 << event.xbutton.button);
+               Vmouse_depressed = Qt;
+               last_mouse_frame = f;
+             }
+           else
+             {
+               x_mouse_grabbed &= ~(1 << event.xbutton.button);
+               if (!x_mouse_grabbed)
+                 Vmouse_depressed = Qnil;
              }
 
            if (numchars >= 1 && emacs_event.kind != no_event)
@@ -3178,6 +4306,10 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
                count++;
                numchars--;
              }
+
+#ifdef USE_X_TOOLKIT
+           goto OTHER;
+#endif /* USE_X_TOOLKIT */
          }
          break;
 
@@ -3205,13 +4337,13 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
          if (numchars >= 2)
            {
              bufp->kind = ascii_keystroke;
-             bufp->code = (char) 'X' & 037; /* C-x */
+             bufp->code = 'X' & 037; /* C-x */
              XSET (bufp->frame_or_window, Lisp_Frame, f);
              XSET (bufp->time, Lisp_Int, event.xkey.time);
              bufp++;
 
              bufp->kind = ascii_keystroke;
-             bufp->code = (char) 0; /* C-@ */
+             bufp->code = 0; /* C-@ */
              XSET (bufp->frame_or_window, Lisp_Frame, f);
              XSET (bufp->time, Lisp_Int, event.xkey.time);
              bufp++;
@@ -3242,37 +4374,81 @@ XTread_socket (sd, bufp, numchars, waitp, expected)
            case MappingKeyboard:
              XRefreshKeyboardMapping (&event.xmapping);
            }
+#ifdef USE_X_TOOLKIT
+         goto OTHER;
+#endif /* USE_X_TOOLKIT */
          break;
 
        default:
+#ifdef USE_X_TOOLKIT
+       OTHER:
+         BLOCK_INPUT;
+         XtDispatchEvent (&event);
+         UNBLOCK_INPUT;
+#endif /* USE_X_TOOLKIT */
          break;
        }
     }
 
-#if 0
+#ifdef X_IO_BUG
+  if (! event_found) 
+    {
+      /* On some systems, an X bug causes Emacs to get no more events
+        when the window is destroyed.  Detect that.  (1994.)  */
+      /* Emacs and the X Server eats up CPU time if XNoOp is done every time.
+        One XNOOP in 100 loops will make Emacs terminate.
+        B. Bretthauer, 1994 */
+      x_noop_count++;
+      if (x_noop_count >= 100) 
+       {
+         x_noop_count=0;
+         XNoOp (x_current_display);
+       }
+    }
+#endif /* X_IO_BUG */
+
+#if 0 /* This fails for serial-line connections to the X server, 
+        because the characters arrive one by one, and a partial
+        command makes select return but gives nothing to read.
+        We'll have to hope that the bug that this tried to fix
+        in 1988 has been fixed in Xlib or the X server.  */
 #ifdef HAVE_SELECT
   if (expected && ! event_found)
     {
       /* AOJ 880406: if select returns true but XPending doesn't, it means that
         there is an EOF condition; in other words, that X has died.
         Act as if there had been a hangup. */
-
       int fd = ConnectionNumber (x_current_display);
-      int mask = 1 << fd;
-
-      if (0 != select (fd + 1, &mask, (long *) 0, (long *) 0,
-                      (EMACS_TIME) 0)
+      SELECT_TYPE mask, junk1, junk2;
+      EMACS_TIME timeout;
+
+      FD_ZERO (&mask);
+      FD_SET (fd, &mask);
+      EMACS_SET_SECS_USECS (timeout, 0, 0);
+      FD_ZERO (&junk1);
+      FD_ZERO (&junk2);
+      if (0 != select (fd + 1, &mask, &junk1, &junk2, &timeout)
          && !XStuffPending ())
        kill (getpid (), SIGHUP);
     }
-#endif /* ! defined (HAVE_SELECT) */
-#endif /* 0 */
+#endif /* HAVE_SELECT */
+#endif /* 0 */
 
 #ifndef HAVE_X11
   if (updating_frame == 0)
     x_do_pending_expose ();
 #endif
 
+  /* If the focus was just given to an autoraising frame,
+     raise it now.  */
+#ifdef HAVE_X11
+  if (pending_autoraise_frame)
+    {
+      x_raise_frame (pending_autoraise_frame);
+      pending_autoraise_frame = 0;
+    }
+#endif
+
   UNBLOCK_INPUT;
   return count;
 }
@@ -3352,10 +4528,10 @@ static void
 x_draw_box (f)
      struct frame *f;
 {
-  int left = CHAR_TO_PIXEL_COL (f, f->cursor_x);
-  int top  = CHAR_TO_PIXEL_ROW (f, f->cursor_y);
+  int left = CHAR_TO_PIXEL_COL (f, curs_x);
+  int top  = CHAR_TO_PIXEL_ROW (f, curs_y);
   int width = FONT_WIDTH (f->display.x->font);
-  int height = FONT_HEIGHT (f->display.x->font);
+  int height = f->display.x->line_height;
 
 #ifdef HAVE_X11
   XDrawRectangle (x_current_display, FRAME_X_WINDOW (f),
@@ -3401,7 +4577,7 @@ clear_cursor (f)
   XPixSet (FRAME_X_WINDOW (f),
           CHAR_TO_PIXEL_COL (f, f->phys_cursor_x),
           CHAR_TO_PIXEL_ROW (f, f->phys_cursor_y),
-          FONT_WIDTH (f->display.x->font), FONT_HEIGHT (f->display.x->font),
+          FONT_WIDTH (f->display.x->font), f->display.x->line_height,
           f->display.x->background_pixel);
 #endif /* ! defined (HAVE_X11) */
   f->phys_cursor_x = -1;
@@ -3421,7 +4597,7 @@ x_draw_single_glyph (f, row, column, glyph, highlight)
   dumpglyphs (f,
              CHAR_TO_PIXEL_COL (f, column),
              CHAR_TO_PIXEL_ROW (f, row),
-             &glyph, 1, highlight);
+             &glyph, 1, highlight, 0);
 }
 
 static void
@@ -3476,7 +4652,7 @@ x_display_bar_cursor (f, on)
                      f->display.x->cursor_gc,
                      CHAR_TO_PIXEL_COL (f, curs_x),
                      CHAR_TO_PIXEL_ROW (f, curs_y),
-                     1, FONT_HEIGHT (f->display.x->font));
+                     1, f->display.x->line_height);
 
       f->phys_cursor_x = curs_x;
       f->phys_cursor_y = curs_y;
@@ -3529,10 +4705,35 @@ x_display_box_cursor (f, on)
          || (f->display.x->current_cursor != hollow_box_cursor
              && (f != x_highlight_frame))))
     {
+      int mouse_face_here = 0;
+
+      /* If the cursor is in the mouse face area, redisplay that when
+        we clear the cursor.  */
+      if (f == mouse_face_mouse_frame
+         &&
+         (f->phys_cursor_y > mouse_face_beg_row
+          || (f->phys_cursor_y == mouse_face_beg_row
+              && f->phys_cursor_x >= mouse_face_beg_col))
+         &&
+         (f->phys_cursor_y < mouse_face_end_row
+          || (f->phys_cursor_y == mouse_face_end_row
+              && f->phys_cursor_x < mouse_face_end_col)))
+       mouse_face_here = 1;
+
+      /* If the font is not as tall as a whole line,
+        we must explicitly clear the line's whole height.  */
+      if (FONT_HEIGHT (f->display.x->font) != f->display.x->line_height)
+       XClearArea (x_current_display, FRAME_X_WINDOW (f),
+                   CHAR_TO_PIXEL_COL (f, f->phys_cursor_x),
+                   CHAR_TO_PIXEL_ROW (f, f->phys_cursor_y),
+                   FONT_WIDTH (f->display.x->font),
+                   f->display.x->line_height, False);
       /* Erase the cursor by redrawing the character underneath it.  */
       x_draw_single_glyph (f, f->phys_cursor_y, f->phys_cursor_x,
                           f->phys_cursor_glyph,
-                          current_glyphs->highlight[f->phys_cursor_y]);
+                          (mouse_face_here
+                           ? 3
+                           : current_glyphs->highlight[f->phys_cursor_y]));
       f->phys_cursor_x = -1;
     }
 
@@ -3573,6 +4774,8 @@ x_display_cursor (f, on)
      struct frame *f;
      int on;
 {
+  BLOCK_INPUT;
+
   if (FRAME_DESIRED_CURSOR (f) == filled_box_cursor)
     x_display_box_cursor (f, on);
   else if (FRAME_DESIRED_CURSOR (f) == bar_cursor)
@@ -3580,6 +4783,8 @@ x_display_cursor (f, on)
   else
     /* Those are the only two we have implemented!  */
     abort ();
+
+  UNBLOCK_INPUT;
 }
 \f
 /* Icons.  */
@@ -3763,7 +4968,7 @@ x_connection_closed ()
   if (_Xdebug)
     abort ();
 
-  shut_down_emacs (0, 1);
+  shut_down_emacs (0, 1, Qnil);
 
   exit (70);
 }
@@ -3785,9 +4990,11 @@ x_error_quitter (display, error)
   fprintf (stderr, "X protocol error: %s on protocol request %d\n",
           buf, error->request_code);
 
+#if 0
   /* While we're testing Emacs 19, we'll just dump core whenever we
      get an X error, so we can figure out why it happened.  */
   abort ();
+#endif
 
   x_connection_closed ();
 }
@@ -3802,15 +5009,18 @@ x_io_error_quitter (display)
   fprintf (stderr, "Connection to X server %s lost.\n",
           XDisplayName (DisplayString (display)));
 
+#if 0
   /* While we're testing Emacs 19, we'll just dump core whenever we
      get an X error, so we can figure out why it happened.  */
   abort ();
+#endif
 
   x_connection_closed ();
 }
 
 /* A buffer for storing X error messages.  */
-static char (*x_caught_error_message)[200];
+static char *x_caught_error_message;
+#define X_CAUGHT_ERROR_MESSAGE_SIZE 200
 
 /* An X error handler which stores the error message in
    x_caught_error_message.  This is what's installed when
@@ -3821,7 +5031,7 @@ x_error_catcher (display, error)
      XErrorEvent *error;
 {
   XGetErrorText (display, error->error_code,
-                *x_caught_error_message, sizeof (*x_caught_error_message));
+                x_caught_error_message, X_CAUGHT_ERROR_MESSAGE_SIZE);
 }
 
 
@@ -3835,7 +5045,7 @@ x_error_catcher (display, error)
 
    Calling x_uncatch_errors resumes the normal error handling.  */
 
-void x_catch_errors(), x_check_errors (), x_uncatch_errors ();
+void x_catch_errors (), x_check_errors (), x_uncatch_errors ();
 
 void
 x_catch_errors ()
@@ -3844,9 +5054,9 @@ x_catch_errors ()
   XSync (x_current_display, False);
 
   /* Set up the error buffer.  */
-  x_caught_error_message =
-    (char (*)[]) xmalloc (sizeof (*x_caught_error_message));
-  (*x_caught_error_message)[0] = '\0';
+  x_caught_error_message
+    = (char*) xmalloc (X_CAUGHT_ERROR_MESSAGE_SIZE);
+  x_caught_error_message[0] = '\0';
 
   /* Install our little error handler.  */
   XHandleError (x_error_catcher);
@@ -3855,6 +5065,7 @@ x_catch_errors ()
 /* If any X protocol errors have arrived since the last call to
    x_catch_errors or x_check_errors, signal an Emacs error using
    sprintf (a buffer, FORMAT, the x error message text) as the text.  */
+
 void
 x_check_errors (format)
      char *format;
@@ -3862,22 +5073,34 @@ x_check_errors (format)
   /* Make sure to catch any errors incurred so far.  */
   XSync (x_current_display, False);
 
-  if ((*x_caught_error_message)[0])
+  if (x_caught_error_message[0])
     {
-      char buf[256];
-
-      sprintf (buf, format, *x_caught_error_message);
-      xfree (x_caught_error_message);
+      char buf[X_CAUGHT_ERROR_MESSAGE_SIZE + 56];
 
+      sprintf (buf, format, x_caught_error_message);
       x_uncatch_errors ();
       error (buf);
     }
 }
 
+/* Nonzero if we had any X protocol errors since we did x_catch_errors.  */
+
+int
+x_had_errors_p ()
+{
+  /* Make sure to catch any errors incurred so far.  */
+  XSync (x_current_display, False);
+
+  return x_caught_error_message[0] != 0;
+}
+
+/* Stop catching X protocol errors and let them make Emacs die.  */
+
 void
 x_uncatch_errors ()
 {
   xfree (x_caught_error_message);
+  x_caught_error_message = 0;
   XHandleError (x_error_quitter);
 }
 
@@ -3898,8 +5121,15 @@ x_trace_wire ()
 
 #ifdef HAVE_X11
 
+struct font_info
+{
+  XFontStruct *font;
+  char *name;
+  char *full_name;
+};
+
 /* A table of all the fonts we have already loaded.  */
-static XFontStruct **x_font_table;
+static struct font_info *x_font_table;
 
 /* The current capacity of x_font_table.  */
 static int x_font_table_size;
@@ -3909,6 +5139,11 @@ static int x_font_table_size;
    0 <= n_fonts <= x_font_table_size.  */
 static int n_fonts;
 
+/* Give frame F the font named FONTNAME as its default font, and
+   return the full name of that font.  FONTNAME may be a wildcard
+   pattern; in that case, we choose some font that fits the pattern.
+   The return value shows which font we chose.  */
+
 Lisp_Object
 x_new_font (f, fontname)
      struct frame *f;
@@ -3922,39 +5157,44 @@ x_new_font (f, fontname)
   /* Get a list of all the fonts that match this name.  Once we
      have a list of matching fonts, we compare them against the fonts
      we already have by comparing font ids.  */
-  font_names = (char **) XListFontsWithInfo (x_current_display, fontname,
-                                            1024, &n_matching_fonts,
-                                            &font_info);
+  font_names = (char **) XListFonts (x_current_display, fontname,
+                                    1024, &n_matching_fonts);
+  /* Apparently it doesn't set n_matching_fonts to zero when it can't
+     find any matches; font_names == 0 is the only clue.  */
+  if (! font_names)
+    n_matching_fonts = 0;
 
-  /* If the server couldn't find any fonts whose named matched fontname,
-     return an error code.  */
-  if (n_matching_fonts == 0)
-    return Qnil;
+  /* Don't just give up if n_matching_fonts is 0.
+     Apparently there's a bug on Suns: XListFontsWithInfo can
+     fail to find a font, but XLoadQueryFont may still find it.  */
 
   /* See if we've already loaded a matching font. */
-  {
-    int i, j;
+  already_loaded = -1;
+  if (n_matching_fonts != 0)
+    {
+      int i, j;
 
-    already_loaded = 0;
-    for (i = 0; i < n_fonts; i++)
-      for (j = 0; j < n_matching_fonts; j++)
-       if (x_font_table[i]->fid == font_info[j].fid)
-         {
-           already_loaded = i;
-           fontname = font_names[j];
-           goto found_font;
-         }
-  }
+      for (i = 0; i < n_fonts; i++)
+       for (j = 0; j < n_matching_fonts; j++)
+         if (!strcmp (x_font_table[i].name, font_names[j])
+             || !strcmp (x_font_table[i].full_name, font_names[j]))
+           {
+             already_loaded = i;
+             fontname = x_font_table[i].full_name;
+             goto found_font;
+           }
+    }
  found_font:
   
   /* If we have, just return it from the table.  */
-  if (already_loaded)
-    f->display.x->font = x_font_table[already_loaded];
+  if (already_loaded >= 0)
+    f->display.x->font = x_font_table[already_loaded].font;
   
   /* Otherwise, load the font and add it to the table.  */
   else
     {
       int i;
+      char *full_name;
       XFontStruct *font;
 
       /* Try to find a character-cell font in the list.  */
@@ -3967,36 +5207,60 @@ x_new_font (f, fontname)
       i = 0;
 #endif
 
-      if (i >= n_matching_fonts)
-       return Qt;
-      else
+      /* See comment above.  */
+      if (n_matching_fonts != 0)
        fontname = font_names[i];
 
       font = (XFontStruct *) XLoadQueryFont (x_current_display, fontname);
       if (! font)
-       return Qnil;
+       {
+         /* Free the information from XListFonts.  */
+         if (n_matching_fonts)
+           XFreeFontNames (font_names);
+         return Qnil;
+       }
 
       /* Do we need to create the table?  */
       if (x_font_table_size == 0)
        {
          x_font_table_size = 16;
          x_font_table
-           = (XFontStruct **) xmalloc (x_font_table_size
-                                       * sizeof (x_font_table[0]));
+           = (struct font_info *) xmalloc (x_font_table_size
+                                           * sizeof (x_font_table[0]));
        }
       /* Do we need to grow the table?  */
       else if (n_fonts >= x_font_table_size)
        {
          x_font_table_size *= 2;
          x_font_table
-           = (XFontStruct **) xrealloc (x_font_table,
-                                        (x_font_table_size
-                                         * sizeof (x_font_table[0])));
+           = (struct font_info *) xrealloc (x_font_table,
+                                            (x_font_table_size
+                                             * sizeof (x_font_table[0])));
        }
 
-      f->display.x->font = x_font_table[n_fonts++] = font;
+      /* Try to get the full name of FONT.  Put it in full_name.  */
+      full_name = 0;
+      for (i = 0; i < font->n_properties; i++)
+       {
+         char *atom
+           = XGetAtomName (x_current_display, font->properties[i].name);
+         if (!strcmp (atom, "FONT"))
+           full_name = XGetAtomName (x_current_display,
+                                     (Atom) (font->properties[i].card32));
+         XFree (atom);
+       }
+
+      x_font_table[n_fonts].name = (char *) xmalloc (strlen (fontname) + 1);
+      bcopy (fontname, x_font_table[n_fonts].name, strlen (fontname) + 1);
+      if (full_name != 0)
+       x_font_table[n_fonts].full_name = full_name;
+      else
+       x_font_table[n_fonts].full_name = x_font_table[n_fonts].name;
+      f->display.x->font = x_font_table[n_fonts++].font = font;
+
+      fontname = full_name;
     }
-  
+
   /* Now make the frame display the given font.  */
   if (FRAME_X_WINDOW (f) != 0)
     {
@@ -4007,16 +5271,22 @@ x_new_font (f, fontname)
       XSetFont (x_current_display, f->display.x->cursor_gc,
                f->display.x->font->fid);
 
-      x_set_window_size (f, f->width, f->height);
+      frame_update_line_height (f);
+      x_set_window_size (f, 0, f->width, f->height);
     }
+  else
+    /* If we are setting a new frame's font for the first time,
+       there are no faces yet, so this font's height is the line height.  */
+    f->display.x->line_height = FONT_HEIGHT (f->display.x->font);
 
   {
-    Lisp_Object lispy_name = build_string (fontname);
+    Lisp_Object lispy_name;
 
+    lispy_name = build_string (fontname);
 
-    /* Free the information from XListFontsWithInfo.  The data
+    /* Free the information from XListFonts.  The data
        we actually retain comes from XLoadQueryFont.  */
-    XFreeFontInfo (font_names, font_info, n_matching_fonts);
+    XFreeFontNames (font_names);
 
     return lispy_name;
   }
@@ -4039,25 +5309,56 @@ x_new_font (f, newname)
   f->display.x->font = temp;
 
   if (FRAME_X_WINDOW (f) != 0)
-    x_set_window_size (f, f->width, f->height);
+    x_set_window_size (f, 0, f->width, f->height);
 
   return 0;
 }
 #endif /* ! defined (HAVE_X11) */
 \f
-/* X Window sizes and positions.  */
-
 x_calc_absolute_position (f)
      struct frame *f;
 {
 #ifdef HAVE_X11
-  if (f->display.x->left_pos < 0)
-    f->display.x->left_pos
-      = x_screen_width - PIXEL_WIDTH (f) + f->display.x->left_pos;
+  Window win, child;
+  int win_x = 0, win_y = 0;
+  int flags = f->display.x->size_hint_flags;
 
-  if (f->display.x->top_pos < 0)
-    f->display.x->top_pos
-      = x_screen_height - PIXEL_HEIGHT (f) + f->display.x->top_pos;
+  /* Find the position of the outside upper-left corner of
+     the inner window, with respect to the outer window.  */
+  if (f->display.x->parent_desc != ROOT_WINDOW)
+    {
+      BLOCK_INPUT;
+      XTranslateCoordinates (x_current_display,
+                              
+                            /* From-window, to-window.  */
+                            f->display.x->window_desc,
+                            f->display.x->parent_desc,
+
+                            /* From-position, to-position.  */
+                            0, 0, &win_x, &win_y,
+
+                            /* Child of win.  */
+                            &child);
+      UNBLOCK_INPUT;
+    }
+
+  /* Treat negative positions as relative to the leftmost bottommost
+     position that fits on the screen.  */
+  if (flags & XNegative)
+    f->display.x->left_pos = (x_screen_width 
+                             - 2 * f->display.x->border_width - win_x
+                             - PIXEL_WIDTH (f)
+                             + f->display.x->left_pos);
+
+  if (flags & YNegative)
+    f->display.x->top_pos = (x_screen_height
+                            - 2 * f->display.x->border_width - win_y
+                            - PIXEL_HEIGHT (f)
+                            + f->display.x->top_pos);
+  /* The left_pos and top_pos
+     are now relative to the top and left screen edges,
+     so the flags should correspond.  */
+  f->display.x->size_hint_flags &= ~ (XNegative | YNegative);
 #else /* ! defined (HAVE_X11) */
   WINDOWINFO_TYPE parentinfo;
 
@@ -4073,45 +5374,87 @@ x_calc_absolute_position (f)
 #endif /* ! defined (HAVE_X11) */
 }
 
-x_set_offset (f, xoff, yoff)
+/* CHANGE_GRAVITY is 1 when calling from Fset_frame_position,
+   to really change the position, and 0 when calling from
+   x_make_frame_visible (in that case, XOFF and YOFF are the current
+   position values).  */
+
+x_set_offset (f, xoff, yoff, change_gravity)
      struct frame *f;
      register int xoff, yoff;
+     int change_gravity;
 {
-  f->display.x->top_pos = yoff;
-  f->display.x->left_pos = xoff;
+  if (change_gravity)
+    {
+      f->display.x->top_pos = yoff;
+      f->display.x->left_pos = xoff;
+      f->display.x->size_hint_flags &= ~ (XNegative | YNegative);
+      if (xoff < 0)
+       f->display.x->size_hint_flags |= XNegative;
+      if (yoff < 0)
+       f->display.x->size_hint_flags |= YNegative;
+      f->display.x->win_gravity = NorthWestGravity;
+    }
   x_calc_absolute_position (f);
 
   BLOCK_INPUT;
+  x_wm_set_size_hint (f, 0, 0);
+
+#ifdef USE_X_TOOLKIT
+  XMoveWindow (XDISPLAY XtWindow (f->display.x->widget),
+              f->display.x->left_pos, f->display.x->top_pos);
+#else /* not USE_X_TOOLKIT */
   XMoveWindow (XDISPLAY FRAME_X_WINDOW (f),
               f->display.x->left_pos, f->display.x->top_pos);
-#ifdef HAVE_X11
-  x_wm_set_size_hint (f, 0);
-#endif /* ! defined (HAVE_X11) */
+#endif /* not USE_X_TOOLKIT */
   UNBLOCK_INPUT;
 }
 
-/* Call this to change the size of frame F's x-window. */
+/* Call this to change the size of frame F's x-window.
+   If CHANGE_GRAVITY is 1, we change to top-left-corner window gravity
+   for this size change and subsequent size changes.
+   Otherwise we leave the window gravity unchanged.  */
 
-x_set_window_size (f, cols, rows)
+x_set_window_size (f, change_gravity, cols, rows)
      struct frame *f;
+     int change_gravity;
      int cols, rows;
 {
   int pixelwidth, pixelheight;
   int mask;
 
+#ifdef USE_X_TOOLKIT
+  BLOCK_INPUT;
+  {
+    /* The x and y position of the widget is clobbered by the
+       call to XtSetValues within EmacsFrameSetCharSize.
+       This is a real kludge, but I don't understand Xt so I can't
+       figure out a correct fix.  Can anyone else tell me? -- rms.  */
+    int xpos = f->display.x->widget->core.x;
+    int ypos = f->display.x->widget->core.y;
+    EmacsFrameSetCharSize (f->display.x->edit_widget, cols, rows);
+    f->display.x->widget->core.x = xpos;
+    f->display.x->widget->core.y = ypos;
+  }
+  UNBLOCK_INPUT;
+
+#else /* not USE_X_TOOLKIT */
+
   BLOCK_INPUT;
 
   check_frame_size (f, &rows, &cols);
-  f->display.x->vertical_scroll_bar_extra =
-    (FRAME_HAS_VERTICAL_SCROLL_BARS (f)
-     ? VERTICAL_SCROLL_BAR_PIXEL_WIDTH (f)
-     : 0);
+  f->display.x->vertical_scroll_bar_extra
+    (FRAME_HAS_VERTICAL_SCROLL_BARS (f)
+       ? VERTICAL_SCROLL_BAR_PIXEL_WIDTH (f)
+       : 0);
   pixelwidth = CHAR_TO_PIXEL_WIDTH (f, cols);
   pixelheight = CHAR_TO_PIXEL_HEIGHT (f, rows);
 
 #ifdef HAVE_X11
-  x_wm_set_size_hint (f, 0);
+  f->display.x->win_gravity = NorthWestGravity;
+  x_wm_set_size_hint (f, 0, 0);
 #endif /* ! defined (HAVE_X11) */
+  XSync (x_current_display, False);
   XChangeWindowSize (FRAME_X_WINDOW (f), pixelwidth, pixelheight);
 
   /* Now, strictly speaking, we can't be sure that this is accurate,
@@ -4128,6 +5471,14 @@ x_set_window_size (f, cols, rows)
   PIXEL_WIDTH (f) = pixelwidth;
   PIXEL_HEIGHT (f) = pixelheight;
 
+  /* If cursor was outside the new size, mark it as off.  */
+  if (f->phys_cursor_y >= rows
+      || f->phys_cursor_x >= cols)
+    {
+      f->phys_cursor_x = -1;
+      f->phys_cursor_y = -1;
+    }
+
   /* We've set {FRAME,PIXEL}_{WIDTH,HEIGHT} to the values we hope to
      receive in the ConfigureNotify event; if we get what we asked
      for, then the event won't cause the screen to become garbaged, so
@@ -4136,6 +5487,7 @@ x_set_window_size (f, cols, rows)
 
   XFlushQueue ();
   UNBLOCK_INPUT;
+#endif /* not USE_X_TOOLKIT */
 }
 
 #ifndef HAVE_X11
@@ -4146,22 +5498,21 @@ x_set_resize_hint (f)
                  2 * f->display.x->internal_border_width,
                  2 * f->display.x->internal_border_width,
                  FONT_WIDTH (f->display.x->font),
-                 FONT_HEIGHT (f->display.x->font));
+                 f->display.x->line_height);
 }
 #endif /* HAVE_X11 */
 \f
 /* Mouse warping, focus shifting, raising and lowering.  */
 
+void
 x_set_mouse_position (f, x, y)
      struct frame *f;
      int x, y;
 {
   int pix_x, pix_y;
 
-  x_raise_frame (f);
-
   pix_x = CHAR_TO_PIXEL_COL (f, x) + FONT_WIDTH  (f->display.x->font) / 2;
-  pix_y = CHAR_TO_PIXEL_ROW (f, y) + FONT_HEIGHT (f->display.x->font) / 2;
+  pix_y = CHAR_TO_PIXEL_ROW (f, y) + f->display.x->line_height / 2;
 
   if (pix_x < 0) pix_x = 0;
   if (pix_x > PIXEL_WIDTH (f)) pix_x = PIXEL_WIDTH (f);
@@ -4175,11 +5526,26 @@ x_set_mouse_position (f, x, y)
   UNBLOCK_INPUT;
 }
 
+/* Move the mouse to position pixel PIX_X, PIX_Y relative to frame F.  */
+
+void
+x_set_mouse_pixel_position (f, pix_x, pix_y)
+     struct frame *f;
+     int pix_x, pix_y;
+{
+  BLOCK_INPUT;
+
+  XWarpMousePointer (FRAME_X_WINDOW (f), pix_x, pix_y);
+  UNBLOCK_INPUT;
+}
+
 #ifdef HAVE_X11
 x_focus_on_frame (f)
      struct frame *f;
 {
+#if 0  /* This proves to be unpleasant.  */
   x_raise_frame (f);
+#endif
 #if 0
   /* I don't think that the ICCCM allows programs to do things like this
      without the interaction of the window manager.  Whatever you end up
@@ -4210,7 +5576,11 @@ x_raise_frame (f)
   if (f->async_visible)
     {
       BLOCK_INPUT;
+#ifdef USE_X_TOOLKIT
+      XRaiseWindow (XDISPLAY XtWindow (f->display.x->widget));
+#else /* not USE_X_TOOLKIT */
       XRaiseWindow (XDISPLAY FRAME_X_WINDOW (f));
+#endif /* not USE_X_TOOLKIT */
       XFlushQueue ();
       UNBLOCK_INPUT;
     }
@@ -4224,7 +5594,11 @@ x_lower_frame (f)
   if (f->async_visible)
     {
       BLOCK_INPUT;
+#ifdef USE_X_TOOLKIT
+      XLowerWindow (XDISPLAY XtWindow (f->display.x->widget));
+#else /* not USE_X_TOOLKIT */
       XLowerWindow (XDISPLAY FRAME_X_WINDOW (f));
+#endif /* not USE_X_TOOLKIT */
       XFlushQueue ();
       UNBLOCK_INPUT;
     }
@@ -4242,7 +5616,8 @@ XTframe_raise_lower (f, raise)
 }
 
 
-/* Change from withdrawn state to mapped state. */
+/* Change from withdrawn state to mapped state,
+   or deiconify. */
 
 x_make_frame_visible (f)
      struct frame *f;
@@ -4254,12 +5629,25 @@ x_make_frame_visible (f)
   if (! FRAME_VISIBLE_P (f))
     {
 #ifdef HAVE_X11
+#ifndef USE_X_TOOLKIT
+      if (! FRAME_ICONIFIED_P (f))
+       x_set_offset (f, f->display.x->left_pos, f->display.x->top_pos, 0);
+#endif
+
       if (! EQ (Vx_no_window_manager, Qt))
        x_wm_set_window_state (f, NormalState);
-
+#ifdef USE_X_TOOLKIT
+      /* This was XtPopup, but that did nothing for an iconified frame.  */
+      XtMapWidget (f->display.x->widget);
+#else /* not USE_X_TOOLKIT */
       XMapWindow (XDISPLAY FRAME_X_WINDOW (f));
+#endif /* not USE_X_TOOLKIT */
+#if 0 /* This seems to bring back scroll bars in the wrong places
+        if the window configuration has changed.  They seem
+        to come back ok without this.  */
       if (FRAME_HAS_VERTICAL_SCROLL_BARS (f))
        XMapSubwindows (x_current_display, FRAME_X_WINDOW (f));
+#endif
 #else /* ! defined (HAVE_X11) */
       XMapWindow (XDISPLAY FRAME_X_WINDOW (f));
       if (f->display.x->icon_desc != 0)
@@ -4277,6 +5665,30 @@ x_make_frame_visible (f)
   XFlushQueue ();
 
   UNBLOCK_INPUT;
+
+  /* Synchronize to ensure Emacs knows the frame is visible
+     before we do anything else.  We do this loop with input not blocked
+     so that incoming events are handled.  */
+  {
+    Lisp_Object frame;
+    int one_in_four = 1;
+    XSET (frame, Lisp_Frame, f);
+    while (! f->async_visible)
+      {
+       x_sync (frame);
+       /* On HPUX on the HP800, the sleep is needed sometimes.  */
+       if ((one_in_four & 3) == 0)
+         {
+#ifdef EMACS_HAS_USECS
+           Fsleep_for (make_number (1), make_number (0));
+#else
+           Fsleep_for (make_number (0), make_number (250));
+#endif
+         }
+       one_in_four++;
+      }
+    FRAME_SAMPLE_VISIBILITY (f);
+  }
 }
 
 /* Change from mapped state to withdrawn state. */
@@ -4285,25 +5697,41 @@ x_make_frame_invisible (f)
      struct frame *f;
 {
   int mask;
+  Window window;
+
+#ifdef USE_X_TOOLKIT
+  /* Use the frame's outermost window, not the one we normally draw on.  */
+  window = XtWindow (f->display.x->widget);
+#else /* not USE_X_TOOLKIT */
+  window = FRAME_X_WINDOW (f);
+#endif /* not USE_X_TOOLKIT */
 
   /* Don't keep the highlight on an invisible frame.  */
   if (x_highlight_frame == f)
     x_highlight_frame = 0;
 
+#if 0/* This might add unreliability; I don't trust it -- rms.  */
   if (! f->async_visible && ! f->async_iconified)
     return;
+#endif
 
   BLOCK_INPUT;
 
+  /* Before unmapping the window, update the WM_SIZE_HINTS property to claim
+     that the current position of the window is user-specified, rather than
+     program-specified, so that when the window is mapped again, it will be
+     placed at the same location, without forcing the user to position it
+     by hand again (they have already done that once for this window.)  */
+  x_wm_set_size_hint (f, 0, 1);
+
 #ifdef HAVE_X11R4
 
-  if (! XWithdrawWindow (x_current_display, FRAME_X_WINDOW (f),
+  if (! XWithdrawWindow (x_current_display, window,
                         DefaultScreen (x_current_display)))
     {
       UNBLOCK_INPUT_RESIGNAL;
-      error ("can't notify window manager of window withdrawl");
+      error ("Can't notify window manager of window withdrawal");
     }
-
 #else /* ! defined (HAVE_X11R4) */
 #ifdef HAVE_X11
 
@@ -4313,7 +5741,7 @@ x_make_frame_invisible (f)
       XEvent unmap;
 
       unmap.xunmap.type = UnmapNotify;
-      unmap.xunmap.window = FRAME_X_WINDOW (f);
+      unmap.xunmap.window = window;
       unmap.xunmap.event = DefaultRootWindow (x_current_display);
       unmap.xunmap.from_configure = False;
       if (! XSendEvent (x_current_display,
@@ -4323,24 +5751,33 @@ x_make_frame_invisible (f)
                        &unmap))
        {
          UNBLOCK_INPUT_RESIGNAL;
-         error ("can't notify window manager of withdrawal");
+         error ("Can't notify window manager of withdrawal");
        }
     }
 
   /* Unmap the window ourselves.  Cheeky!  */
-  XUnmapWindow (x_current_display, FRAME_X_WINDOW (f));
-
+  XUnmapWindow (x_current_display, window);
 #else /* ! defined (HAVE_X11) */
 
   XUnmapWindow (FRAME_X_WINDOW (f));
-  f->async_visible = 0;                /* Handled by the UnMap event for X11 */
   if (f->display.x->icon_desc != 0)
     XUnmapWindow (f->display.x->icon_desc);
 
 #endif /* ! defined (HAVE_X11) */
 #endif /* ! defined (HAVE_X11R4) */
 
-  XFlushQueue ();
+  /* We can't distinguish this from iconification
+     just by the event that we get from the server.
+     So we can't win using the usual strategy of letting
+     FRAME_SAMPLE_VISIBILITY set this.  So do it by hand,
+     and synchronize with the server to make sure we agree.  */
+  f->visible = 0;
+  FRAME_ICONIFIED_P (f) = 0;
+  f->async_visible = 0;
+  f->async_iconified = 0;
+
+  x_sync ();
+
   UNBLOCK_INPUT;
 }
 
@@ -4350,6 +5787,7 @@ x_iconify_frame (f)
      struct frame *f;
 {
   int mask;
+  int result;
 
   /* Don't keep the highlight on an invisible frame.  */
   if (x_highlight_frame == f)
@@ -4358,9 +5796,42 @@ x_iconify_frame (f)
   if (f->async_iconified)
     return;
 
+#ifdef USE_X_TOOLKIT
+  BLOCK_INPUT;
+
+  if (! FRAME_VISIBLE_P (f))
+    {
+      if (! EQ (Vx_no_window_manager, Qt))
+       x_wm_set_window_state (f, IconicState);
+      /* This was XtPopup, but that did nothing for an iconified frame.  */
+      XtMapWidget (f->display.x->widget);
+      UNBLOCK_INPUT;
+      return;
+    }
+
+  result = XIconifyWindow (x_current_display,
+                          XtWindow (f->display.x->widget),
+                          DefaultScreen (x_current_display));
+  UNBLOCK_INPUT;
+
+  if (!result)
+    error ("Can't notify window manager of iconification");
+
+  f->async_iconified = 1;
+
+  BLOCK_INPUT;
+  XFlushQueue ();
+  UNBLOCK_INPUT;
+#else /* not USE_X_TOOLKIT */
+
   BLOCK_INPUT;
 
 #ifdef HAVE_X11
+  /* Make sure the X server knows where the window should be positioned,
+     in case the user deiconifies with the window manager.  */
+  if (! FRAME_VISIBLE_P (f) && !FRAME_ICONIFIED_P (f))
+    x_set_offset (f, f->display.x->left_pos, f->display.x->top_pos, 0);
+
   /* Since we don't know which revision of X we're running, we'll use both
      the X11R3 and X11R4 techniques.  I don't know if this is a good idea.  */
 
@@ -4382,7 +5853,7 @@ x_iconify_frame (f)
                      &message))
       {
        UNBLOCK_INPUT_RESIGNAL;
-       error ("Can't notify window manager of iconification.");
+       error ("Can't notify window manager of iconification");
       }
   }
 
@@ -4390,6 +5861,16 @@ x_iconify_frame (f)
      IconicState.  */
   x_wm_set_window_state (f, IconicState);
 
+  if (!FRAME_VISIBLE_P (f))
+    {
+      /* If the frame was withdrawn, before, we must map it.  */
+      XMapWindow (XDISPLAY FRAME_X_WINDOW (f));
+#if 0 /* We don't have subwindows in the icon.  */
+      if (FRAME_HAS_VERTICAL_SCROLL_BARS (f))
+       XMapSubwindows (x_current_display, FRAME_X_WINDOW (f));
+#endif
+    }
+
   f->async_iconified = 1;
 #else /* ! defined (HAVE_X11) */
   XUnmapWindow (XDISPLAY FRAME_X_WINDOW (f));
@@ -4404,6 +5885,7 @@ x_iconify_frame (f)
 
   XFlushQueue ();
   UNBLOCK_INPUT;
+#endif /* not USE_X_TOOLKIT */
 }
 
 /* Destroy the X window of frame F.  */
@@ -4416,6 +5898,11 @@ x_destroy_window (f)
   if (f->display.x->icon_desc != 0)
     XDestroyWindow (XDISPLAY f->display.x->icon_desc);
   XDestroyWindow (XDISPLAY f->display.x->window_desc);
+#ifdef USE_X_TOOLKIT
+  XtDestroyWidget (f->display.x->widget);
+  free_frame_menubar (f);
+#endif /* USE_X_TOOLKIT */
+
   free_frame_faces (f);
   XFlushQueue ();
 
@@ -4426,6 +5913,13 @@ x_destroy_window (f)
   if (f == x_highlight_frame)
     x_highlight_frame = 0;
 
+  if (f == mouse_face_mouse_frame)
+    {
+      mouse_face_beg_row = mouse_face_beg_col = -1;
+      mouse_face_end_row = mouse_face_end_col = -1;
+      mouse_face_window = Qnil;
+    }
+
   UNBLOCK_INPUT;
 }
 \f
@@ -4445,7 +5939,7 @@ x_destroy_window (f)
    of certain kinds into our private queues.
 
    All ExposeRegion events are put in x_expose_queue.
-   All ButtonPressed and ButtonReleased events are put in x_mouse_queue.  */
+   All ButtonPress and ButtonRelease events are put in x_mouse_queue.  */
 
 
 /* Write the event *P_XREP into the event queue *QUEUE.
@@ -4507,68 +6001,100 @@ mouse_event_pending_p ()
 
 #ifdef HAVE_X11
 
-x_wm_set_size_hint (f, prompting)
+/* Set the normal size hints for the window manager, for frame F.
+   FLAGS is the flags word to use--or 0 meaning preserve the flags
+   that the window now has.
+   If USER_POSITION is nonzero, we set the USPosition
+   flag (this is useful when FLAGS is 0).  */
+
+x_wm_set_size_hint (f, flags, user_position)
      struct frame *f;
-     long prompting;
+     long flags;
+     int user_position;
 {
   XSizeHints size_hints;
+
+#ifdef USE_X_TOOLKIT
+  Arg al[2];
+  int ac = 0;
+  Dimension widget_width, widget_height;
+  Window window = XtWindow (f->display.x->widget);
+#else /* not USE_X_TOOLKIT */
   Window window = FRAME_X_WINDOW (f);
+#endif /* not USE_X_TOOLKIT */
 
+  /* Setting PMaxSize caused various problems.  */
   size_hints.flags = PResizeInc | PMinSize /* | PMaxSize */;
 
   flexlines = f->height;
 
   size_hints.x = f->display.x->left_pos;
   size_hints.y = f->display.x->top_pos;
+
+#ifdef USE_X_TOOLKIT
+  XtSetArg (al[ac], XtNwidth, &widget_width); ac++;
+  XtSetArg (al[ac], XtNheight, &widget_height); ac++;
+  XtGetValues (f->display.x->column_widget, al, ac);
+  size_hints.height = widget_height;
+  size_hints.width = widget_width;
+#else /* not USE_X_TOOLKIT */
   size_hints.height = PIXEL_HEIGHT (f);
   size_hints.width = PIXEL_WIDTH (f);
+#endif /* not USE_X_TOOLKIT */
+
   size_hints.width_inc = FONT_WIDTH (f->display.x->font);
-  size_hints.height_inc = FONT_HEIGHT (f->display.x->font);
-#if 0
+  size_hints.height_inc = f->display.x->line_height;
   size_hints.max_width = x_screen_width - CHAR_TO_PIXEL_WIDTH (f, 0);
   size_hints.max_height = x_screen_height - CHAR_TO_PIXEL_HEIGHT (f, 0);
-#endif    
+
   {
     int base_width, base_height;
+    int min_rows = 0, min_cols = 0;
 
     base_width = CHAR_TO_PIXEL_WIDTH (f, 0);
     base_height = CHAR_TO_PIXEL_HEIGHT (f, 0);
 
-    {
-      int min_rows = 0, min_cols = 0;
-      check_frame_size (f, &min_rows, &min_cols);
-
-      /* The window manager uses the base width hints to calculate the
-        current number of rows and columns in the frame while
-        resizing; min_width and min_height aren't useful for this
-        purpose, since they might not give the dimensions for a
-        zero-row, zero-column frame.
+    check_frame_size (f, &min_rows, &min_cols);
 
-        We use the base_width and base_height members if we have
-        them; otherwise, we set the min_width and min_height members
-        to the size for a zero x zero frame.  */
+    /* The window manager uses the base width hints to calculate the
+       current number of rows and columns in the frame while
+       resizing; min_width and min_height aren't useful for this
+       purpose, since they might not give the dimensions for a
+       zero-row, zero-column frame.
+       
+       We use the base_width and base_height members if we have
+       them; otherwise, we set the min_width and min_height members
+       to the size for a zero x zero frame.  */
 
 #ifdef HAVE_X11R4
-      size_hints.flags |= PBaseSize;
-      size_hints.base_width = base_width;
-      size_hints.base_height = base_height;
-      size_hints.min_width  = base_width + min_cols * size_hints.width_inc;
-      size_hints.min_height = base_height + min_rows * size_hints.height_inc;
+    size_hints.flags |= PBaseSize;
+    size_hints.base_width = base_width;
+    size_hints.base_height = base_height;
+    size_hints.min_width  = base_width + min_cols * size_hints.width_inc;
+    size_hints.min_height = base_height + min_rows * size_hints.height_inc;
 #else
-      size_hints.min_width = base_width;
-      size_hints.min_height = base_height;
+    size_hints.min_width = base_width;
+    size_hints.min_height = base_height;
 #endif
-    }
-
   }
 
-  if (prompting)
-    size_hints.flags |= prompting;
+  if (flags)
+    size_hints.flags |= flags;
   else
     {
       XSizeHints hints;                /* Sometimes I hate X Windows... */
+      long supplied_return;
+      int value;
+
+#ifdef HAVE_X11R4
+      value = XGetWMNormalHints (x_current_display, window, &hints,
+                                &supplied_return);
+#else
+      value = XGetNormalHints (x_current_display, window, &hints);
+#endif
       
-      XGetNormalHints (x_current_display, window, &hints);
+      if (value == 0)
+       hints.flags = 0;
       if (hints.flags & PSize)
        size_hints.flags |= PSize;
       if (hints.flags & PPosition)
@@ -4579,6 +6105,17 @@ x_wm_set_size_hint (f, prompting)
        size_hints.flags |= USSize;
     }
 
+#ifdef PWinGravity
+  size_hints.win_gravity = f->display.x->win_gravity;
+  size_hints.flags |= PWinGravity;
+
+  if (user_position)
+    {
+      size_hints.flags &= ~ PPosition;
+      size_hints.flags |= USPosition;
+    }
+#endif /* PWinGravity */
+
 #ifdef HAVE_X11R4
   XSetWMNormalHints (x_current_display, window, &size_hints);
 #else
@@ -4591,19 +6128,30 @@ x_wm_set_window_state (f, state)
      struct frame *f;
      int state;
 {
+#ifdef USE_X_TOOLKIT
+  Arg al[1];
+
+  XtSetArg (al[0], XtNinitialState, state);
+  XtSetValues (f->display.x->widget, al, 1);
+#else /* not USE_X_TOOLKIT */
   Window window = FRAME_X_WINDOW (f);
 
   f->display.x->wm_hints.flags |= StateHint;
   f->display.x->wm_hints.initial_state = state;
 
   XSetWMHints (x_current_display, window, &f->display.x->wm_hints);
+#endif /* not USE_X_TOOLKIT */
 }
 
 x_wm_set_icon_pixmap (f, icon_pixmap)
      struct frame *f;
      Pixmap icon_pixmap;
 {
+#ifdef USE_X_TOOLKIT
+  Window window = XtWindow (f->display.x->widget);
+#else
   Window window = FRAME_X_WINDOW (f);
+#endif
 
   if (icon_pixmap)
     {
@@ -4620,7 +6168,11 @@ x_wm_set_icon_position (f, icon_x, icon_y)
      struct frame *f;
      int icon_x, icon_y;
 {
+#ifdef USE_X_TOOLKIT
+  Window window = XtWindow (f->display.x->widget);
+#else
   Window window = FRAME_X_WINDOW (f);
+#endif
 
   f->display.x->wm_hints.flags |= IconPositionHint;
   f->display.x->wm_hints.icon_x = icon_x;
@@ -4632,53 +6184,86 @@ x_wm_set_icon_position (f, icon_x, icon_y)
 \f
 /* Initialization.  */
 
+#ifdef USE_X_TOOLKIT
+static XrmOptionDescRec emacs_options[] = {
+  {"-geometry",        ".geometry", XrmoptionSepArg, NULL},
+  {"-iconic",  ".iconic", XrmoptionNoArg, (XtPointer) "yes"},
+
+  {"-internal-border-width", "*EmacsScreen.internalBorderWidth",
+     XrmoptionSepArg, NULL},
+  {"-ib",      "*EmacsScreen.internalBorderWidth", XrmoptionSepArg, NULL},
+
+  {"-T",       "*EmacsShell.title", XrmoptionSepArg, (XtPointer) NULL},
+  {"-wn",      "*EmacsShell.title", XrmoptionSepArg, (XtPointer) NULL},
+  {"-title",   "*EmacsShell.title", XrmoptionSepArg, (XtPointer) NULL},
+  {"-iconname",        "*EmacsShell.iconName", XrmoptionSepArg, (XtPointer) NULL},
+  {"-in",      "*EmacsShell.iconName", XrmoptionSepArg, (XtPointer) NULL},
+  {"-mc",      "*pointerColor", XrmoptionSepArg, (XtPointer) NULL},
+  {"-cr",      "*cursorColor", XrmoptionSepArg, (XtPointer) NULL}
+};
+#endif /* USE_X_TOOLKIT */
+
 void
-x_term_init (display_name)
+x_term_init (display_name, xrm_option, resource_name)
      char *display_name;
+     char *xrm_option;
+     char *resource_name;
 {
   Lisp_Object frame;
   char *defaultvalue;
+  int argc = 0;
+  char** argv = 0;
+#ifndef F_SETOWN_BUG
 #ifdef F_SETOWN
   extern int old_fcntl_owner;
 #endif /* ! defined (F_SETOWN) */
+#endif /* F_SETOWN_BUG */
   
+#ifdef X_IO_BUG
+  x_noop_count = 0;
+#endif
+
   x_focus_frame = x_highlight_frame = 0;
 
+#ifdef USE_X_TOOLKIT
+  argv = (char **) XtMalloc (7 * sizeof (char *));
+  argv[0] = "";
+  argv[1] = "-display";
+  argv[2] = display_name;
+  argv[3] = "-name";
+  /* Usually `emacs', but not always.  */
+  argv[4] = resource_name;
+  argc = 5;
+  if (xrm_option)
+    {
+      argv[argc++] = "-xrm";
+      argv[argc++] = xrm_option;
+    }
+  Xt_app_shell = XtAppInitialize (&Xt_app_con, "Emacs",
+                                 emacs_options, XtNumber (emacs_options),
+                                 &argc, argv,
+                                 NULL, NULL, 0);
+  XtFree ((char *)argv);
+  x_current_display = XtDisplay (Xt_app_shell);
+
+#else /* not USE_X_TOOLKIT */
   x_current_display = XOpenDisplay (display_name);
+#endif /* not USE_X_TOOLKIT */
   if (x_current_display == 0)
-    fatal ("X server %s not responding; check the DISPLAY environment variable or use \"-d\"\n",
+    fatal ("X server %s not responding.\n\
+Check the DISPLAY environment variable or use \"-d\"\n",
           display_name);
 
 #ifdef HAVE_X11
   {
-    int hostname_size = 256;
-
-    hostname = (char *) xmalloc (hostname_size);
-
 #if 0
     XSetAfterFunction (x_current_display, x_trace_wire);
 #endif /* ! 0 */
-
-    /* Try to get the host name; if the buffer is too short, try
-       again.  Apparently, the only indication gethostname gives of
-       whether the buffer was large enough is the presence or absence
-       of a '\0' in the string.  Eech.  */
-    for (;;)
-      {
-       gethostname (hostname, hostname_size - 1);
-       hostname[hostname_size - 1] = '\0';
-
-       /* Was the buffer large enough for gethostname to store the '\0'?  */
-       if (strlen (hostname) < hostname_size - 1)
-         break;
-
-       hostname_size <<= 1;
-       hostname = (char *) xrealloc (hostname, hostname_size);
-      }
     x_id_name = (char *) xmalloc (XSTRING (Vinvocation_name)->size
-                               + strlen (hostname)
+                               + XSTRING (Vsystem_name)->size
                                + 2);
-    sprintf (x_id_name, "%s@%s", XSTRING (Vinvocation_name)->data, hostname);
+    sprintf (x_id_name, "%s@%s",
+            XSTRING (Vinvocation_name)->data, XSTRING (Vsystem_name)->data);
   }
 
   /* Figure out which modifier bits mean what.  */
@@ -4695,40 +6280,27 @@ x_term_init (display_name)
 #endif
 
   if (ConnectionNumber (x_current_display) != 0)
-    {
-      dup2 (ConnectionNumber (x_current_display), 0);
-
-#ifndef SYSV_STREAMS
-      /* Streams somehow keeps track of which descriptor number
-        is being used to talk to X.  So it is not safe to substitute
-        descriptor 0.  But it is safe to make descriptor 0 a copy of it.  */
-      close (ConnectionNumber (x_current_display));
-      ConnectionNumber (x_current_display) = 0;        /* Looks a little strange?
-                                                * check the def of the macro;
-                                                * it is a genuine lvalue */
-#endif /* SYSV_STREAMS */
-    }
+    change_keyboard_wait_descriptor (ConnectionNumber (x_current_display));
+  change_input_fd (ConnectionNumber (x_current_display));
 
 #endif /* ! defined (HAVE_X11) */
   
+#ifndef F_SETOWN_BUG
 #ifdef F_SETOWN
-  old_fcntl_owner = fcntl (0, F_GETOWN, 0);
+  old_fcntl_owner = fcntl (ConnectionNumber (x_current_display), F_GETOWN, 0);
 #ifdef F_SETOWN_SOCK_NEG
-  fcntl (0, F_SETOWN, -getpid ());     /* stdin is a socket here */
+  /* stdin is a socket here */
+  fcntl (ConnectionNumber (x_current_display), F_SETOWN, -getpid ());
 #else /* ! defined (F_SETOWN_SOCK_NEG) */
-  fcntl (0, F_SETOWN, getpid ());
+  fcntl (ConnectionNumber (x_current_display), F_SETOWN, getpid ());
 #endif /* ! defined (F_SETOWN_SOCK_NEG) */
 #endif /* ! defined (F_SETOWN) */
+#endif /* F_SETOWN_BUG */
 
 #ifdef SIGIO
   init_sigio ();
 #endif /* ! defined (SIGIO) */
 
-  /* Must use interrupt input because we cannot otherwise
-     arrange for C-g to be noticed immediately.
-     We cannot connect it to SIGINT.  */
-  Fset_input_mode (Qt, Qnil, Qt, Qnil);
-
   expose_all_windows = 0;
 
   clear_frame_hook = XTclear_frame;
@@ -4745,6 +6317,7 @@ x_term_init (display_name)
   update_end_hook = XTupdate_end;
   set_terminal_window_hook = XTset_terminal_window;
   read_socket_hook = XTread_socket;
+  frame_up_to_date_hook = XTframe_up_to_date;
   cursor_to_hook = XTcursor_to;
   reassert_line_highlight_hook = XTreassert_line_highlight;
   mouse_position_hook = XTmouse_position;
@@ -4763,6 +6336,9 @@ x_term_init (display_name)
                                   off the bottom */
   baud_rate = 19200;
 
+  /* Try to use interrupt input; if we can't, then start polling.  */
+  Fset_input_mode (Qt, Qnil, Qt, Qnil);
+
   /* Note that there is no real way portable across R3/R4 to get the 
      original error handler.  */
   XHandleError (x_error_quitter);
@@ -4780,6 +6356,9 @@ void
 syms_of_xterm ()
 {
   staticpro (&last_mouse_scroll_bar);
+  last_mouse_scroll_bar = Qnil;
+  staticpro (&mouse_face_window);
+  mouse_face_window = Qnil;
 }
 #endif /* ! defined (HAVE_X11) */
 #endif /* ! defined (HAVE_X_WINDOWS) */