apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <wr...@rowe-clan.net>
Subject Re: [PROPOSAL] cgi_exec_info_t: detached & addrspace fields combined
Date Tue, 29 Jun 2004 18:29:41 GMT
At 04:17 AM 6/28/2004, Joe Orton wrote:
>OK, the apr_procattr_addrspace_set() interface is sufficient to solve
>this problem, right?  And there's no issue with back-porting that to the
>APR 0.9 branch?  The only issue is how to use that interface from
>mod_cgi/the Netware MPM without requiring an httpd major MMN bump?  So
>why not just overload the 'detached' field in cgi_exec_info_t inside
>mod_cgi/Netware MPM?  That's should cause too much damange.

Agreed, if that's a private structure, no bumps needed.

>-1 on overloading apr_procattr_detach_set() to do this, that's bad API
>design, and uglifying the APR API just to satisfy an httpd binary compat
>requirement just seems very wrong.

I agree.  If you want to overload the bit flags, that's fine, because the actual
apr_procattr_t should be opaque to the user, correct?

But having two seperate fn's to set detached v.s. addrspace seems like
a much better public API.

View raw message