Mercurial > emacs

changeset 70058:12c8842436ef

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
(window_loop): Test w->dedicated with !NILP instead of EQ Qt. (window_scroll_pixel_based): Fix off-by-one bug in 2002-12-23 change.
author Kim F. Storm <storm@cua.dk>
date Mon, 17 Apr 2006 21:50:59 +0000
parents 08c02110e935
children 9028ccec5e38
files src/window.c
diffstat 1 files changed, 9 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/src/window.c	Mon Apr 17 21:50:43 2006 +0000
+++ b/src/window.c	Mon Apr 17 21:50:59 2006 +0000
@@ -662,6 +662,8 @@
 	{
 	  if (!WINDOW_LEFTMOST_P (w) && abs (*x - x0) < grabbable_width)
 	    {
+	      /* Convert X and Y to window relative coordinates.
+		 Vertical border is at the left edge of window.  */
 	      *x = max (0, *x - x0);
 	      *y -= top_y;
 	      return ON_VERTICAL_BORDER;
@@ -671,6 +673,8 @@
 	{
 	  if (abs (*x - x1) < grabbable_width)
 	    {
+	      /* Convert X and Y to window relative coordinates.
+		 Vertical border is at the right edge of window.  */
 	      *x = min (x1, *x) - x0;
 	      *y -= top_y;
 	      return ON_VERTICAL_BORDER;
@@ -717,6 +721,8 @@
 	  && !WINDOW_RIGHTMOST_P (w)
 	  && (abs (*x - right_x) < grabbable_width))
 	{
+	  /* Convert X and Y to window relative coordinates.
+	     Vertical border is at the right edge of window.  */
 	  *x = min (right_x, *x) - left_x;
 	  *y -= top_y;
 	  return ON_VERTICAL_BORDER;
@@ -2027,7 +2033,7 @@
 	       `obj & 1' means consider only full-width windows.
 	       `obj & 2' means consider also dedicated windows. */
 	    if (((XINT (obj) & 1) && !WINDOW_FULL_WIDTH_P (w))
-		|| (!(XINT (obj) & 2) && EQ (w->dedicated, Qt))
+		|| (!(XINT (obj) & 2) && !NILP (w->dedicated))
 		/* Minibuffer windows are always ignored.  */
 		|| MINI_WINDOW_P (w))
 	      break;
@@ -2082,7 +2088,7 @@
 	  case GET_LARGEST_WINDOW:
 	    { /* nil `obj' means to ignore dedicated windows.  */
 	      /* Ignore dedicated windows and minibuffers.  */
-	      if (MINI_WINDOW_P (w) || (NILP (obj) && EQ (w->dedicated, Qt)))
+	      if (MINI_WINDOW_P (w) || (NILP (obj) && !NILP (w->dedicated)))
 		break;
 
 	      if (NILP (best_window))
@@ -4868,7 +4874,7 @@
 	{
 	  if (it.current_y < it.last_visible_y
 	      && (it.current_y + it.max_ascent + it.max_descent
-		  >= it.last_visible_y))
+		  > it.last_visible_y))
 	    {
 	      /* The last line was only partially visible, make it fully
 		 visible.  */