GNU bug report logs -
#11464
24.1.50; pos-visible-in-window-p returns a false positive with bidi text
Previous Next
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
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.