Re: abort () is scm_gc_sweep ()

[Marius Vollmer]

Ludovic Courtès <address@hidden> writes:

> In function scm_gc_sweep () (gc.c:1922) appears the following condition:
>
>   if (scm_mallocated < m)
>     abort ();
>
> The comment above the abort () is very clear about why this happens.
> However, since the triggering of this abort () can only be due to the
> non respect of Guile's programming conventions by the programmer, I
> think it would be a good idea to rather use something like
>
>   assert (scm_mallocated < m);
>
> instead of just abort ().

Yes, that would be better.  There are a lot of places where we use
abort.  Maybe most of them could be changed to assertions.

So, if someone has a patch... ;-)

-- 
GPG: D5D4E405 - 2F9B BCCC 8527 692A 04E3  331E FAF8 226A D5D4 E405