Re: [jimb@red-bean.com: Redisplay bug in 21.2.90.1]

[storm@cua.dk (Kim F. Storm)]

Richard Stallman <rms@gnu.org> writes:

>     I just committed a fix on HEAD.
> 
> Thanks.
> 
>     I guess it isn't serious enough to include on RC branch...?
> 
> Maybe it is--it depends how safe the fix is. 

I think the fix is quite safe.

I actually added the fix in three different functions which
inserts text in a buffer.  Only one of these funtions was 
responsible for the behaviour reported by Jim.


> Could you show us the fix?
> 

Index: insdel.c
===================================================================
RCS file: /cvs/emacs/src/insdel.c,v
retrieving revision 1.166
retrieving revision 1.167
diff -c -r1.166 -r1.167
*** insdel.c	9 Feb 2002 22:57:25 -0000	1.166
--- insdel.c	13 Jun 2002 22:07:13 -0000	1.167
***************
*** 1031,1036 ****
--- 1031,1040 ----
    if (GPT_BYTE < GPT)
      abort ();
  
+   /* The insert may have been in the unchanged region, so check again. */
+   if (Z - GPT < END_UNCHANGED)
+     END_UNCHANGED = Z - GPT;
+ 
    adjust_overlays_for_insert (PT, nchars);
    adjust_markers_for_insert (PT, PT_BYTE,
  			     PT + nchars, PT_BYTE + nbytes,
***************
*** 1154,1159 ****
--- 1158,1167 ----
    if (GPT_BYTE < GPT)
      abort ();
  
+   /* The insert may have been in the unchanged region, so check again. */
+   if (Z - GPT < END_UNCHANGED)
+     END_UNCHANGED = Z - GPT;
+ 
    adjust_overlays_for_insert (PT, nchars);
    adjust_markers_for_insert (PT, PT_BYTE, PT + nchars,
  			     PT_BYTE + outgoing_nbytes,
***************
*** 1300,1305 ****
--- 1308,1317 ----
  
    if (GPT_BYTE < GPT)
      abort ();
+ 
+   /* The insert may have been in the unchanged region, so check again. */
+   if (Z - GPT < END_UNCHANGED)
+     END_UNCHANGED = Z - GPT;
  
    adjust_overlays_for_insert (PT, nchars);
    adjust_markers_for_insert (PT, PT_BYTE, PT + nchars,

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk