Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Errors-To: <wrowe@rowe-clan.net>
Message-ID: <04ee01c113c4$d90adc90$93c0b0d0@roweclan.net>
From: "William A. Rowe, Jr." <wrowe@rowe-clan.net>
To: <trawick@attglobal.net>, <dev@apr.apache.org>
References: <200107232011.QAA32338@smtp-server.nc.rr.com>
Subject: Re: apr_XXX_set_inherit() brokenness
Date: Mon, 23 Jul 2001 17:09:55 -0500
MIME-Version: 1.0
Content-Type: text/plain;
	charset="Windows-1252"
Content-Transfer-Encoding: 7bit

The Crux of the problem is in the disconnected way the entire cleanup API
is working.

apr_pool_child_cleanup_kill() is pure BS, since it also kills the plain cleanup.
Nobody uses it, so I'm killing the function.

I'll replace it with a apr_pool_child_cleanup_set() that allows you to replace
the original child cleanup with a new cleanup, given the void* and plain cleanup.

Bill

----- Original Message ----- 
From: "Jeff Trawick" <trawick@attglobal.net>
To: <dev@apr.apache.org>
Sent: Monday, July 23, 2001 3:11 PM
Subject: apr_XXX_set_inherit() brokenness


> two problems, I think:
> 
> 1) The special cleanup needs to be removed if the apr_foo is closed
>    before the pool goes away.
> 
>    Example:
> 
>      mod_file_cache/mmap-ed file
> 
>      open file
>      create mmap
>      close file
> 
>      The inherit cleanup is still in place at this point.
> 
> 2) Either there should be a no-op plain cleanup or the close-pool code
>    should not call plain_cleanup if that ptr is NULL.
> 
>    run_cleanups() unconditionally calls cleanup->plain_cleanup(), so
>    this tries to branch to zero if this is the inherit cleanup.
> 
> -- 
> Jeff Trawick | trawick@attglobal.net | PGP public key at web site:
>        http://www.geocities.com/SiliconValley/Park/9289/
>              Born in Roswell... married an alien...
>