changeset 109905:c2d849aa4021

Fix mirroring pixel positions on GUI terminals. dispnew.c (buffer_posn_from_coords): Fix off-by-one error in mirroring pixel positions.
author Eli Zaretskii <eliz@gnu.org>
date Sat, 21 Aug 2010 12:35:31 +0300
parents bb6ea55bfdc0
children 4083ea4ca585 5a930b6b595e
files src/ChangeLog src/dispnew.c
diffstat 2 files changed, 16 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/src/ChangeLog	Sat Aug 21 00:46:23 2010 -0400
+++ b/src/ChangeLog	Sat Aug 21 12:35:31 2010 +0300
@@ -1,3 +1,8 @@
+2010-08-21  Eli Zaretskii  <eliz@gnu.org>
+
+	* dispnew.c (buffer_posn_from_coords): Fix off-by-one error in
+	mirroring pixel positions.
+
 2010-08-20  Dan Nicolaescu  <dann@ics.uci.edu>
 
 	* alloc.c (malloc_sbrk_used, malloc_sbrk_unused): Remove,
--- a/src/dispnew.c	Sat Aug 21 00:46:23 2010 -0400
+++ b/src/dispnew.c	Sat Aug 21 12:35:31 2010 +0300
@@ -5386,12 +5386,12 @@
 
   /* First, move to the beginning of the row corresponding to *Y.  We
      need to be in that row to get the correct value of base paragraph
-     direction for the paragraph at *X.  */
+     direction for the text at (*X, *Y).  */
   move_it_to (&it, -1, 0, *y, -1, MOVE_TO_X | MOVE_TO_Y);
 
   /* TO_X is the pixel position that the iterator will compute for the
-     glyph at *X.  This is because iterator positions are not offset
-     due to hscroll.  */
+     glyph at *X.  We add it.first_visible_x because iterator
+     positions include the hscroll.  */
   to_x = x0 + it.first_visible_x;
   if (it.bidi_it.paragraph_dir == R2L)
     /* For lines in an R2L paragraph, we need to mirror TO_X wrt the
@@ -5401,13 +5401,14 @@
        display, we reverse their order in PRODUCE_GLYPHS, but the
        iterator doesn't know about that.)  The following line adjusts
        the pixel position to the iterator geometry, which is what
-       move_it_* routines use.  */
-    to_x = window_box_width (w, TEXT_AREA) - to_x
-	   /* Text terminals need a one-character offset to get it right.  */
-	   - (FRAME_MSDOS_P (WINDOW_XFRAME (w))
-	      || FRAME_TERMCAP_P (WINDOW_XFRAME (w)));
-
-  /* Now move horizontally in the row to the glyph under *X. */
+       move_it_* routines use.  (The -1 is because in a window whose
+       text-area width is W, the rightmost pixel position is W-1, and
+       it should be mirrored into zero pixel position.)  */
+    to_x = window_box_width (w, TEXT_AREA) - to_x - 1;
+
+  /* Now move horizontally in the row to the glyph under *X.  Second
+     argument is ZV to prevent move_it_in_display_line from matching
+     based on buffer positions.  */
   move_it_in_display_line (&it, ZV, to_x, MOVE_TO_X);
 
   Fset_buffer (old_current_buffer);