GNU bug report logs - #11464
24.1.50; pos-visible-in-window-p returns a false positive with bidi text

Previous Next

Package: emacs;

Reported by: Ari Roponen <ari.roponen <at> gmail.com>

Date: Sun, 13 May 2012 15:56:01 UTC

Severity: normal

Found in version 24.1.50

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Ari Roponen <ari.roponen <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 11464 <at> debbugs.gnu.org
Subject: bug#11464: 24.1.50; pos-visible-in-window-p returns a false positive with bidi text
Date: Tue, 15 May 2012 13:07:08 +0300
Eli Zaretskii <eliz <at> gnu.org> writes:

>
> But since the fix is quite simple, here you go: fixed in revision
> 107994 on the emacs-24 branch.
>

Thank you. I can still reproduce the bug with that revision, but the
following tweak seems to help.  I'm not sure if it is correct, but at
least it fixes the testcase, and everything else seems to work okay.

=== modified file 'src/xdisp.c'
--- src/xdisp.c	2012-05-13 18:22:35 +0000
+++ src/xdisp.c	2012-05-15 09:51:45 +0000
@@ -1313,7 +1313,7 @@
 	visible_p = bottom_y > window_top_y;
       else if (top_y < it.last_visible_y)
 	visible_p = 1;
-      if (bottom_y >= it.last_visible_y
+      if (bottom_y <= it.last_visible_y
 	  && it.bidi_p && it.bidi_it.scan_dir == -1
 	  && IT_CHARPOS (it) < charpos)
 	{


-- 
Ari Roponen




This bug report was last modified 13 years and 4 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.