develooper Front page | perl.perl5.porters | Postings from May 2002

Re: [PATCH] Storable (Re: perl@16433)

Thread Previous | Thread Next
From:
Jarkko Hietaniemi
Date:
May 7, 2002 06:54
Subject:
Re: [PATCH] Storable (Re: perl@16433)
Message ID:
20020507165342.C12565@alpha.hut.fi
On Tue, May 07, 2002 at 11:37:03AM +0300, Radu Greab wrote:
> On Mon, 6 May 2002 17:22 -0700, Gurusamy Sarathy wrote:
>  > On Mon, 06 May 2002 23:21:47 BST, Nicholas Clark wrote:
>  > >As Storable already passes a context object around to make itself thread
>  > >safe,
>  > 
>  > This "non-standard" context object (really just an old iteration of
>  > the MY_CXT stuff) should be reworked to use the standard MY_CXT macros
>  > that use an SV to allocate their space.
> 
> Teaching Storable the standard MY_CXT macros may be too much work
> because Storable uses contexts not only for thread-safeness, but also
> for recursive invocations. So, Storable needs to manage more contexts
> per interpreter and the MY_CXT macros may not be powerfull enough to
> support this situation.
> 
> With the patch below all tests pass and the leaks are gone. Following
> the MY_CXT macros example all contexts use now a blessed SV and the
> root context is cleaned automatically.

Thanks, applied (and the STMT_XXX one, too).  Now I have to find
other things for Nicholas to do... :-)

-- 
$jhi++; # http://www.iki.fi/jhi/
        # There is this special biologist word we use for 'stable'.
        # It is 'dead'. -- Jack Cohen

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About