apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe Jr." <wr...@apache.org>
Subject Re: svn commit: r930533 - /apr/apr/trunk/include/apr_support.h
Date Wed, 07 Apr 2010 11:31:44 GMT
By rights, we should be moving apr_support.h out of include/, that was a first
mistake.  I have the same understanding as you, Mladen, but the authors of the
httpd mod_reqtimeout seem to be reaching their fingers into this header.

On 4/7/2010 6:29 AM, Mladen Turk wrote:
> Hi,
> 
> Will you do something about that or do I have
> to make the official -1?
> 
> So far you didn't even consider to reply which
> is I presume caused by the email noise.
> 
> 
> On 04/04/2010 05:58 PM, Mladen Turk wrote:
>> On 04/03/2010 05:43 PM, bjh@apache.org wrote:
>>> Author: bjh
>>> Date: Sat Apr 3 15:43:00 2010
>>> New Revision: 930533
>>>
>>> URL: http://svn.apache.org/viewvc?rev=930533&view=rev
>>> Log:
>>> Add missing APR_DECLARE for apr_wait_for_io_or_timeout() declaration.
>>>
>>
>> I don't think this was missing.
>> apr_wait_for_io_or_timeout was not a public API which you
>> have just made.
>>
>> See what apr_support.h says:
>> @defgroup apr_support Internal APR support functions
>>
> 
> Regards


Mime
View raw message