Re: [PATCH 2/2] ruby: enable garbage collection using talloc

[David Bremner <david@tethera.net>]

Felipe Contreras <felipe.contreras@gmail.com> writes:

> We basically steal all the objects from their notmuch parents, therefore
> they are completely under Ruby's gc control.
>
> The order at which these objects are freed does not matter any more,
> because destroying the database does not destroy all the children
> objects, since they belong to Ruby now.
>

I guess from a purist point of view this is a kind of layering
violation, since the use of talloc is purportedly an internal
implementation detail of the library. Still, I think it's a reasonable
approach given that the ruby bindings are maintained as part of notmuch,
and we are not very likely to abandon talloc.

I am OK with applying the series as is. The only sensible thing I can
think of at the moment for testing is to run something like the script
of id:20210517193915.1220114-1-felipe.contreras@gmail.com as a "time
test", so not attempt to get valgrind working, but just run it on some
decent size corpuses and see that it it does not crash or leak too much
memory to complete.

d