Geoffrey Young <[EMAIL PROTECTED]> writes:

[...]

> >   ./file_io/win32/filestat.c:            
   apr_pool_cleanup_register(finfo->pool, pdesc, free_localheap,
> 
> that is certainly an interesting use :)

Nah-  The pool in question comes from apr_stat, which
is supplied by the user.  It's only used above to
postpone the LocalFree()s on all the GetSecurityInfo 
objects.  more_finfo could have been written more clearly
with an extra pool argument, instead of getting the original
argument from finfo->pool.

IMO it makes much more sense to have a read-only pool() 
method for APR::File than it does to expose finfo->pool.
apr_finfo_t doesn't have any semantics outside of being
a return value of some C function.  I'm not sure why there's
even a perl API for it in the first place.

-- 
Joe Schaefer


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

Reply via email to