(reseat_at_next_visible_line_start): Position before

newline only if ending up on a newline.
(next_element_from_ellipsis): Return success.  Handle case of
displaying no ellipsis.  Fix case of ellipsis defined in display
table.
(next_element_from_buffer): Return 0 if next_element_from_ellipsis
returns 0.

diff --git a/src/xdisp.c b/src/xdisp.c
index ddce21eb951432f2c9e96527de1a56e146a7132e..09c9b70f506fc7f5ac01ca54facfc211d130e278 100644 (file)
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -3132,7 +3132,9 @@ reseat_at_next_visible_line_start (it, on_newline_p)
          forward_to_next_line_start (it);
 
       /* Position on the newline if we should.  */
-      if (on_newline_p && IT_CHARPOS (*it) > BEGV)
+      if (on_newline_p
+         && IT_CHARPOS (*it) > BEGV
+         && FETCH_BYTE (IT_BYTEPOS (*it) - 1) == '\n')
        {
          --IT_CHARPOS (*it);
          IT_BYTEPOS (*it) = CHAR_TO_BYTE (IT_CHARPOS (*it));
@@ -3831,32 +3833,37 @@ next_element_from_c_string (it)
    entry.  This function Fills IT with the first glyph from the
    ellipsis if an ellipsis is to be displayed.  */
 
-static void
+static int
 next_element_from_ellipsis (it)
      struct it *it;
 {
-  if (it->dp && VECTORP (DISP_INVIS_VECTOR (it->dp)))
+  if (it->selective_display_ellipsis_p)
     {
-      /* Use the display table definition for `...'.  Invalid glyphs
-        will be handled by the method returning elements from dpvec.  */
-      struct Lisp_Vector *v = XVECTOR (DISP_INVIS_VECTOR (it->dp));
-      it->dpvec_char_len = it->len;
-      it->dpvec = v->contents;
-      it->dpend = v->contents + v->size;
-      it->current.dpvec_index = 0;
-      it->method = next_element_from_display_vector;
-      get_next_display_element (it);
-    }
-  else if (it->selective_display_ellipsis_p)
-    {
-      /* Use default `...' which is stored in default_invis_vector.  */
-      it->dpvec_char_len = it->len;
-      it->dpvec = default_invis_vector;
-      it->dpend = default_invis_vector + 3;
-      it->current.dpvec_index = 0;
-      it->method = next_element_from_display_vector;
-      get_next_display_element (it);
+      if (it->dp && VECTORP (DISP_INVIS_VECTOR (it->dp)))
+       {
+         /* Use the display table definition for `...'.  Invalid glyphs
+            will be handled by the method returning elements from dpvec.  */
+         struct Lisp_Vector *v = XVECTOR (DISP_INVIS_VECTOR (it->dp));
+         it->dpvec_char_len = it->len;
+         it->dpvec = v->contents;
+         it->dpend = v->contents + v->size;
+         it->current.dpvec_index = 0;
+         it->method = next_element_from_display_vector;
+       }
+      else
+       {
+         /* Use default `...' which is stored in default_invis_vector.  */
+         it->dpvec_char_len = it->len;
+         it->dpvec = default_invis_vector;
+         it->dpend = default_invis_vector + 3;
+         it->current.dpvec_index = 0;
+         it->method = next_element_from_display_vector;
+       }
     }
+  else
+    reseat_at_next_visible_line_start (it, 1);
+  
+  return get_next_display_element (it);
 }
 
 
@@ -3977,7 +3984,7 @@ next_element_from_buffer (it)
                                        IT_BYTEPOS (*it) + 1,
                                        it->selective))
                {
-                 next_element_from_ellipsis (it);
+                 success_p = next_element_from_ellipsis (it);
                  it->dpvec_char_len = -1;
                }
            }
@@ -3986,7 +3993,7 @@ next_element_from_buffer (it)
              /* A value of selective == -1 means that everything from the
                 CR to the end of the line is invisible, with maybe an
                 ellipsis displayed for it.  */
-             next_element_from_ellipsis (it);
+             success_p = next_element_from_ellipsis (it);
              it->dpvec_char_len = -1;
            }
        }