Re: removing scm_gc_mark_conservatively()

[Marius Vollmer]

[ Sorry for coming so late. ]

Dirk Herrmann <address@hidden> writes:

> On Sun, 16 Sep 2001, Chris Cramer wrote:
> 
> > This patch (for stable) removes scm_gc_mark_conservatively().
> > It fixes all the segfault problems I've been having.
> 
> Applied (with two minor changes) to stable and development.  Thanks for
> your patch and the bug hunting!

Shouldn't you also always set the cell type to scm_tc16_allocated in
SCM_NEWCELL?  Else you might miss references during code like

    SCM_NEWCELL (z);
    SCM_SET_CELL_OBJECT_1 (z, x);
    scm_gc ();  // is not scanned here and x might no longer be living.
    SCM_SET_CELL_WORD_0 (z, scm_tc16_whatever);

I'm not sure whether we have such code, but we might.