You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apr.apache.org by Jeff Trawick <tr...@gmail.com> on 2005/11/11 21:40:32 UTC
Re: Proper declaration for apr_wait_for_io_or_timeout() (was: Re: NWGNUmakefile update for APR1.2?)
On 11/11/05, Brad Nicholes <BN...@novell.com> wrote:
> Reposting to the APR list with a new subject line. Does this need to
> be taken care of in APR 1.2.x before httpd 2.2 ships?
apr_wait_for_io_or_timeout() is private APR function (not API). Is
the caller outside of APR?
Re: Proper declaration for apr_wait_for_io_or_timeout() (was: Re: NWGNUmakefile update for APR1.2?)
Posted by Brian Pane <br...@apache.org>.
On Nov 11, 2005, at 6:21 PM, Jeff Trawick wrote:
> On 11/11/05, Brad Nicholes <BN...@novell.com> wrote:
>>
>>>>> On 11/11/2005 at 1:40:32 pm, in message
>> <cc...@mail.gmail.com>, Jeff
>> Trawick
>> <tr...@gmail.com> wrote:
>>> On 11/11/05, Brad Nicholes <BN...@novell.com> wrote:
>>>> Reposting to the APR list with a new subject line. Does this
>>>> need
>> to
>>>> be taken care of in APR 1.2.x before httpd 2.2 ships?
>>>
>>> apr_wait_for_io_or_timeout() is private APR function (not API). Is
>>> the caller outside of APR?
>>
>> Yes, HTTPD. trunk/server/core_filters.c
>
> Simply shocking ;) That code needs to be fixed to call a real API :(
I just double-checked the declaration, and apr_wait_for_io_or_timeout()
currently is implemented as part of the public API. I don't have any
strong
objection to making it private, though (e.g., by wrapping the
declaration in
an "#ifdef APR_PRIVATE" or moving apr_support.h to apr/include/arch/
unix).
Brian
Re: Proper declaration for apr_wait_for_io_or_timeout() (was: Re: NWGNUmakefile update for APR1.2?)
Posted by Jeff Trawick <tr...@gmail.com>.
On 11/11/05, Brad Nicholes <BN...@novell.com> wrote:
>
> >>> On 11/11/2005 at 1:40:32 pm, in message
> <cc...@mail.gmail.com>, Jeff
> Trawick
> <tr...@gmail.com> wrote:
> > On 11/11/05, Brad Nicholes <BN...@novell.com> wrote:
> >> Reposting to the APR list with a new subject line. Does this need
> to
> >> be taken care of in APR 1.2.x before httpd 2.2 ships?
> >
> > apr_wait_for_io_or_timeout() is private APR function (not API). Is
> > the caller outside of APR?
>
> Yes, HTTPD. trunk/server/core_filters.c
Simply shocking ;) That code needs to be fixed to call a real API :(
Re: Proper declaration for apr_wait_for_io_or_timeout() (was:
Re: NWGNUmakefile update for APR1.2?)
Posted by Brad Nicholes <BN...@novell.com>.
>>> On 11/11/2005 at 1:40:32 pm, in message
<cc...@mail.gmail.com>, Jeff
Trawick
<tr...@gmail.com> wrote:
> On 11/11/05, Brad Nicholes <BN...@novell.com> wrote:
>> Reposting to the APR list with a new subject line. Does this need
to
>> be taken care of in APR 1.2.x before httpd 2.2 ships?
>
> apr_wait_for_io_or_timeout() is private APR function (not API). Is
> the caller outside of APR?
Yes, HTTPD. trunk/server/core_filters.c
Brad