Subject: bug#11199: 24.0.95; killing right-to-left text at eob leads to inconsistent state

From: Eli Zaretskii <[email-masked]>
Date: Mon, 09 Apr 2012 15:34:40 +0300

> From: Michael Welsh Duggan <[email-masked]>
> Cc: [email-masked],  [email-masked]
> Date: Mon, 09 Apr 2012 08:18:10 -0400
> 
> > === modified file 'src/xdisp.c'
> > --- src/xdisp.c	2012-03-31 19:30:53 +0000
> > +++ src/xdisp.c	2012-04-09 11:46:50 +0000
> > @@ -16602,7 +16602,15 @@ find_last_unchanged_at_beg_row (struct w
> >  	     continued.  */
> >  	  && !(MATRIX_ROW_END_CHARPOS (row) == first_changed_pos
> >  	       && (row->continued_p
> > -		   || row->exact_window_width_line_p)))
> > +		   || row->exact_window_width_line_p))
> > +	  /* If ROW->end is beyond ZV, then ROW->end is outdated and
> > +	     needs to be recomputed, so don't consider this row as
> > +	     unchanged.  This happens when the last line was
> > +	     bidi-reordered and was killed immediately before this
> > +	     redisplay cycle.  In that case, ROW->end stores the
> > +	     buffer position of the first visual-order character of
> > +	     the next row, which is now beyond ZV.  */
> > +	  && CHARPOS (row->end.pos) <= ZV)
> >  	row_found = row;
> >  
> >        /* Stop if last visible row.  */
> >
> 
> I can verify that this patch keeps the original recipe from causing
> Emacs to crash.  Everything looks good for now.

Thanks, I installed this as revision 107792 on the emacs-24 branch.  I
will wait for Yamamoto-san to confirm that this fixes his problem as
well, before I close this bug.

Thank you both for your great help in solving this bug.