Stas Bekman <[EMAIL PROTECTED]> writes:

[...]

> You were saying delete (which is remove+destroy):
> http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=109285355117298&w=2
> 
>        $b->delete; # APR::Bucket needs to wrap apr_bucket_delete(),
>                    # since APR_BUCKET_REMOVE() doesn't actually
>                    # decrement the bucket refcount
>                    # so the bucket allocator can reclaim $b
> 
> and I've missed the difference between delete and remove. My bad.
> 
> So yes, we need to provide a delete() wrapper and use that.

Which takes us full circle, because my caution against calling
$bb->next($b) *after* calling $b->delete is far more serious, 
because with delete, the underlying apr_bucket_t may have been 
free()d.

> And hopefully one day someone will resolve the problems with
> $bb->cleanup on win32.

+1 - this would make life much simpler once it actually works there.

-- 
Joe Schaefer


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to