Re: [hober0@gmail.com: Re: mode-line redisplay bug]

[YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp>]

>>>>> On Mon, 10 Oct 2005 20:40:18 +0100, Jason Rumney <jasonr@gnu.org> said:

> "Jan D." <jan.h.d@swipnet.se> writes:
>> AFAIK co-ordinates on W32 behave the same as on X, so your patch
>> should be OK.  It is easy to confirm anyway.

> OK, I've installed the patch, but as YAMAMOTO Mitsuharu pointed out,
> it may not deal with the left fringe width - which might be an
> explanation for the gx calculation being off by one, and possibly
> also the explanation for why this patch does work as I expected.

A simple debug code below shows the incorrectness of the calculated
rectangle :-).  And if it is corrected, the problem I said in
http://lists.gnu.org/archive/html/emacs-pretest-bug/2005-06/msg00148.html
will appear.

>>>>> On Tue, 07 Jun 2005 18:45:48 +0900, YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp> said:

>      2.1 Sometimes a tooltip is not shown.  I hope the first
>          fragment of the patch below fix this.

>      2.2 The value of last_mouse_glyph may become invalid.  For
>          example, after clicking the image on the splash screen,
>          dragging the area where the image was displayed does not
>          issue mouse-movement events.  I think last_mouse_glyph
>          should be cleared in some appropriate timing, but I'm not
>          sure when it is.

				     YAMAMOTO Mitsuharu
				mituharu@math.s.chiba-u.ac.jp

Index: src/xterm.c
===================================================================
RCS file: /cvsroot/emacs/emacs/src/xterm.c,v
retrieving revision 1.879
diff -c -r1.879 xterm.c
*** src/xterm.c	10 Oct 2005 22:54:19 -0000	1.879
--- src/xterm.c	11 Oct 2005 01:10:58 -0000
***************
*** 3675,3680 ****
--- 3677,3688 ----
  	      rect->height = r->height;
  	      rect->x = WINDOW_TO_FRAME_PIXEL_X (w, gx);
  	      rect->y = WINDOW_TO_FRAME_PIXEL_Y (w, r->y);
+ 
+ 	      XDrawRectangle (FRAME_X_DISPLAY (f), FRAME_X_WINDOW (f),
+ 			      f->output_data.x->normal_gc,
+ 			      rect->x - 1, rect->y - 1,
+ 			      rect->width + 2, rect->height + 2);
+ 
  	      return 1;
  	    }
  	  break;