From xemacs-m  Tue Mar 11 03:42:01 1997
Received: from crystal.WonderWorks.COM (crystal.WonderWorks.com [192.203.206.1])
	by xemacs.org (8.8.5/8.8.5) with ESMTP id DAA02991
	for <xemacs-beta@xemacs.org>; Tue, 11 Mar 1997 03:41:58 -0600 (CST)
Received: by crystal.WonderWorks.COM 
	id QQcgju19801; Tue, 11 Mar 1997 04:41:57 -0500 (EST)
Date: Tue, 11 Mar 1997 04:41:57 -0500 (EST)
Message-Id: <QQcgju19801.199703110941@crystal.WonderWorks.COM>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
From: Kyle Jones <kyle_jones@wonderworks.com>
To: xemacs-beta@xemacs.org
Subject: Re: 20.1-b6: assertion failed, file extents.c, line 1588
In-Reply-To: <QQcgjh12354.199703110621@crystal.WonderWorks.COM>
References: <199703110353.TAA16754@carnifex.cogit.com>
	<QQcgjf11929.199703110551@crystal.WonderWorks.COM>
	<m2g1y3j6bs.fsf@altair.xemacs.org>
	<QQcgjh12354.199703110621@crystal.WonderWorks.COM>
X-Face: /cA45WHG7jWq>(O3&Z57Y<"WsX5ddc,4c#w0F*zrV#=M
        0@~@,s;b,aMtR5Sqs"+nU.z^CSFQ9t`z2>W,S,]:[+2^
        Nbf6v4g>!&,7R4Ot4Wg{&tm=WX7P["9%a)_da48-^tGy
        ,qz]Z,Zz\{E.,]'EO+F)@$KtF&V

Kyle Jones writes:
 > Steven L Baur writes:
 >  > Kyle Jones writes:
 >  > 
 >  > > This is a rather alarming crash.  Has the extents code been
 >  > > modified recently?
 >  > 
 >  > None since 20.1-b2.  The third previous edit was to fix the crash you
 >  > reported with duplicable, replicating extents (prior to 20.0final).
 >  > The second previous edit was a Bill Perry edit to remove some code
 >  > that enabled a bomb with a text property of Qnil (prior to 20.1-b1).
 >  > The previous edit (prior to 20.1-b2) was to remove the behavior of
 >  > allowing any editing of read-only extents.
 > 
 > Evidence suggests that this crash is related to the new time.el
 > somehow.  The only place I can find set-extent-parent being
 > called is in the redisplay code related to the modeline.  And
 > guess what code has been fiddling with the modeline recently.

Actually the crash is much simpler.

(setq e (make-extent nil nil))
(delete-extent e)
(set-extent-face e 'bold)

Sayonara...

Here's a patch:

*** 1.1	1997/03/11 09:28:47
--- src/extents.c	1997/03/11 09:29:19
***************
*** 4768,4778 ****
  */
         (extent, face))
  {
!   EXTENT e;
    Lisp_Object orig_face = face;
  
-   CHECK_EXTENT (extent);
-   e = XEXTENT (extent);
    /* retrieve the ancestor for efficiency and proper redisplay noting. */
    e = extent_ancestor (e);
  
--- 4768,4776 ----
  */
         (extent, face))
  {
!   EXTENT e = decode_extent(extent, 0);
    Lisp_Object orig_face = face;
  
    /* retrieve the ancestor for efficiency and proper redisplay noting. */
    e = extent_ancestor (e);
  



