# HG changeset patch
# User Chong Yidong <cyd@stupidchicken.com>
# Date 1225084537 0
# Node ID d03ac65823d18b2178a50c7eebd4f2923fde6e39
# Parent  162fccf6d2c3298d3c1e19feb9e5786ab78d85ad
(try_scrolling): When computing distance from the scroll margin, don't
use MOVE_TO_Y.

diff -r 162fccf6d2c3 -r d03ac65823d1 src/xdisp.c
--- a/src/xdisp.c	Mon Oct 27 05:15:17 2008 +0000
+++ b/src/xdisp.c	Mon Oct 27 05:15:37 2008 +0000
@@ -12622,7 +12622,7 @@
   struct text_pos pos, startp;
   struct it it;
   int this_scroll_margin, scroll_max, rc, height;
-  int dy = 0, amount_to_scroll = 0, scroll_down_p = 0;
+  int dy = 0, amount_to_scroll = 0;
   int extra_scroll_margin_lines = last_line_misfit ? 1 : 0;
   Lisp_Object aggressive;
   int scroll_limit = INT_MAX / FRAME_LINE_HEIGHT (f);
@@ -12687,8 +12687,7 @@
 	     window, or below.  Compute the distance from the scroll
 	     margin to PT, and give up if the distance is greater than
 	     scroll_max.  */
-	  move_it_to (&it, PT, -1, it.last_visible_y - 1, -1,
-		      MOVE_TO_POS | MOVE_TO_Y);
+	  move_it_to (&it, PT, -1, -1, -1, MOVE_TO_POS);
 
 	  /* To make point visible, we must move the window start down
 	     so that the cursor line is visible, which means we have
@@ -12697,18 +12696,11 @@
 
 	  if (dy > scroll_max)
 	    return SCROLLING_FAILED;
-
-	  scroll_down_p = 1;
-	}
-    }
-
-  if (scroll_down_p)
-    {
-      /* Move the window start down.  If scrolling conservatively,
-	 move it just enough down to make point visible.  If
-	 scroll_step is set, move it down by scroll_step.  */
-      start_display (&it, w, startp);
-
+	}
+    }
+
+  if (dy > 0)
+    {
       if (scroll_conservatively)
 	/* Set AMOUNT_TO_SCROLL to at least one line,
 	   and at most scroll_conservatively lines.  */
@@ -12733,10 +12725,14 @@
       if (amount_to_scroll <= 0)
 	return SCROLLING_FAILED;
 
+      /* Move the window start down.  If scrolling conservatively,
+	 move it just enough down to make point visible.  If
+	 scroll_step is set, move it down by scroll_step.  */
+      start_display (&it, w, startp);
+      move_it_vertically (&it, amount_to_scroll);
+
       /* If moving by amount_to_scroll leaves STARTP unchanged,
 	 move it down one screen line.  */
-
-      move_it_vertically (&it, amount_to_scroll);
       if (CHARPOS (it.current.pos) == CHARPOS (startp))
 	move_it_by_lines (&it, 1, 1);
       startp = it.current.pos;