Fix occasional recentering under scroll-conservatively.

 xdisp.c (try_scrolling): Compute the limit for searching point
 in forward scroll from scroll_max, instead of an arbitrary limit
 of 10 screen lines.  See
 http://lists.gnu.org/archive/html/emacs-devel/2010-06/msg00766.html and
 http://lists.gnu.org/archive/html/emacs-devel/2010-06/msg00773.html for
 details.

2 files changed, 22 insertions, 7 deletions

diff --git a/src/ChangeLog b/src/ChangeLog
index 9e115942d39..7d875bf5bda 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,13 @@
+2010-06-19  Eli Zaretskii  <eliz@gnu.org>
+
+        * xdisp.c (try_scrolling): Compute the limit for searching point
+        in forward scroll from scroll_max, instead of an arbitrary limit
+        of 10 screen lines.  See
+        http://lists.gnu.org/archive/html/emacs-devel/2010-06/msg00766.html
+        and
+        http://lists.gnu.org/archive/html/emacs-devel/2010-06/msg00773.html
+        for details.
+
 2010-06-16  Glenn Morris  <rgm@gnu.org>
 
         * editfns.c (Fbyte_to_string): Pacify compiler.
diff --git a/src/xdisp.c b/src/xdisp.c
index c8043308ec8..c111ca2ffa3 100644
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -13431,14 +13431,19 @@ try_scrolling (window, just_this_one_p, scroll_conservatively,
       if (PT > CHARPOS (it.current.pos))
         {
           int y0 = line_bottom_y (&it);
-
-          /* Compute the distance from the scroll margin to PT
-             (including the height of the cursor line).  Moving the
-             iterator unconditionally to PT can be slow if PT is far
-             away, so stop 10 lines past the window bottom (is there a
-             way to do the right thing quickly?).  */
+          /* Compute how many pixels below window bottom to stop searching
+             for PT.  This avoids costly search for PT that is far away if
+             the user limited scrolling by a small number of lines, but
+             always finds PT if scroll_conservatively is set to a large
+             number, such as most-positive-fixnum.  */
+          int slack = min (scroll_max, 10 * FRAME_LINE_HEIGHT (f));
+
+          /* Compute the distance from the scroll margin to PT or to
+             the scroll limit, whichever comes first.  This should
+             include the height of the cursor line, to make that line
+             fully visible.  */
           move_it_to (&it, PT, -1,
-                      it.last_visible_y + 10 * FRAME_LINE_HEIGHT (f),
+                      it.last_visible_y + slack,
                       -1, MOVE_TO_POS | MOVE_TO_Y);
           dy = line_bottom_y (&it) - y0;