Re: remainder of patch for DIR_MERGE leak

[Doug MacEachern <do...@covalent.net>]

On Thu, 23 Nov 2000, Matt Sergeant wrote:

> I think this is also required (I've only just gotten around to testing
> it):

if you take that out, from what i can see, cleanups will not be run for
{DIR,SERVER}CREATE.
the patch that went into 1.24_01 fixes the leak for {DIR,SERVER}MERGE.
 
> *** lib/Apache/ExtUtils.pm.old	Thu Nov 23 14:40:04 2000
> --- lib/Apache/ExtUtils.pm	Thu Nov 23 14:40:17 2000
> *************** static mod_perl_perl_dir_config *newPerl
> *** 195,201 ****
>   	    palloc(p, sizeof (mod_perl_perl_dir_config));
>       cld->obj = Nullsv;
>       cld->pclass = "$class";
> -     register_cleanup(p, cld, perl_perl_cmd_cleanup, null_cleanup);
>       return cld;
>   }
>   
> --- 195,200 ----
> 
> At least with my very bad knowledge of mod_perl internals, I think that is
> required...
> 
> -- 
> <Matt/>
> 
>     /||    ** Director and CTO **
>    //||    **  AxKit.com Ltd   **  ** XML Application Serving **
>   // ||    ** http://axkit.org **  ** XSLT, XPathScript, XSP  **
>  // \\| // **     Personal Web Site: http://sergeant.org/     **
>      \\//
>      //\\
>     //  \\
>