Re: Subtle bug in intervals code

[Eli Zaretskii]

> Date: Wed, 18 Jul 2012 15:48:08 +0400
> From: Dmitry Antipov <address@hidden>
> 
> Commit 109118 (by me) introduces the following change in src/intervals.c, 
> function delete_interval:
> 
> @@ -1262,8 +1198,7 @@
>     register INTERVAL parent;
>     ptrdiff_t amt = LENGTH (i);
> 
> -  if (amt > 0)                       /* Only used on zero-length intervals 
> now.  */
> -    abort ();
> +  eassert (amt == 0);                /* Only used on zero-length intervals 
> now.  */
> 
>     if (ROOT_INTERVAL_P (i))
>       {
> 
> Now this eassert traps at interval with negative length. I'm not familiar with
> this subsystem enough to find (possible) bug quickly, so any help is 
> appreciated.

Do you have an easy test case that triggers this?  The only ones
posted are to bootstrap, which is not my idea of an easy test case...