From: Chong Yidong <cyd@stupidchicken.com>
Date: Thu, 7 Aug 2008 01:54:06 +0000 (+0000)
Subject: (redisplay_window): Revert last change.
X-Git-Tag: emacs-pretest-23.0.90~3587
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=c6aecbbd026dbdf263b2da00b4f351556bbd8042;p=emacs.git

(redisplay_window): Revert last change.
(try_window): Check bottom scroll margin too.
---

diff --git a/src/xdisp.c b/src/xdisp.c
index 4c099653a6d..920e7c1d34e 100644
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -13668,11 +13668,6 @@ redisplay_window (window, just_this_one_p)
       debug_method_add (w, "same window start");
 #endif
 
-      /* If there's a scroll margin, we must try to scroll, in case
-	 point is now in the scroll margin.  */
-      if (scroll_margin > 0)
-	goto try_to_scroll;
-
       /* Try to redisplay starting at same place as before.
          If point has not moved off frame, accept the results.  */
       if (!current_matrix_up_to_date_p
@@ -14079,9 +14074,13 @@ try_window (window, pos, check_margins)
     {
       int this_scroll_margin;
 
-      this_scroll_margin = max (0, scroll_margin);
-      this_scroll_margin = min (this_scroll_margin, WINDOW_TOTAL_LINES (w) / 4);
-      this_scroll_margin *= FRAME_LINE_HEIGHT (it.f);
+      if (scroll_margin > 0)
+	{
+	  this_scroll_margin = min (scroll_margin, WINDOW_TOTAL_LINES (w) / 4);
+	  this_scroll_margin *= FRAME_LINE_HEIGHT (f);
+	}
+      else
+	this_scroll_margin = 0;
 
       if ((w->cursor.y >= 0	/* not vscrolled */
 	   && w->cursor.y < this_scroll_margin
@@ -14091,7 +14090,7 @@ try_window (window, pos, check_margins)
 	     seems to give wrong results.  We don't want to recenter
 	     when the last line is partly visible, we want to allow
 	     that case to be handled in the usual way.  */
-	  || (w->cursor.y + 1) > it.last_visible_y)
+	  || w->cursor.y > it.last_visible_y - this_scroll_margin - 1)
 	{
 	  w->cursor.vpos = -1;
 	  clear_glyph_matrix (w->desired_matrix);