Re: module GC bug

[Han-Wen Nienhuys <hanwen@xs4all.nl>]

Mikael Djurfeldt wrote:
> On 7/7/05, Marius Vollmer <mvo@zagadka.de> wrote:
> 
>>>I've fixed this by introducing a new function (eval-closure-module)
>>>which returns the module of a closure via the eval-closure smob.
>>
>>I think the right fix is to change the weak hashtable marking
>>algorithm to properly cope with circular references like this.  I will
>>try this and then come back to you.  (I don't know how long this might
>>take since the required changes look to be slightly non-trivial (but
>>entirely possible).)
> 
> 
> That would be good, because the current fix (which is already
> committed to CVS HEAD, and which removes the setting of the
> procedure-property) still leaves code in modules.c which expects the
> procedure-property to be set.  If Han-Wen's fix is to be kept, then
> you should, at least, remove that code. The consequence is that it
> will no longer be possible to lookup the module of the eval closure
> for any kind of closures except the standard ones (which are
> implemented as a smob).

Hi,

I guess I'm missing something.  Where is the 'module procedure property 
set for other eval closures?

-- 
  Han-Wen Nienhuys - hanwen@xs4all.nl - http://www.xs4all.nl/~hanwen


_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel