You are correct. Fixed in r1734281.
> Am 09.03.2016 um 16:57 schrieb Ruediger Pluem <rpl...@apache.org>:
>
>
>
> On 03/09/2016 01:39 PM, ic...@apache.org wrote:
>> Author: icing
>> Date: Wed Mar 9 12:39:04 2016
>> New Revision: 1734231
>>
>> URL: http://svn.apache.org/viewvc?rev=1734231&view=rev
>> Log:
>> added AP_DECLARE for new ap_create_request
>>
>> Modified:
>> httpd/httpd/trunk/include/http_protocol.h
>>
>> Modified: httpd/httpd/trunk/include/http_protocol.h
>> URL:
>> http://svn.apache.org/viewvc/httpd/httpd/trunk/include/http_protocol.h?rev=1734231&r1=1734230&r2=1734231&view=diff
>> ==============================================================================
>> --- httpd/httpd/trunk/include/http_protocol.h (original)
>> +++ httpd/httpd/trunk/include/http_protocol.h Wed Mar 9 12:39:04 2016
>> @@ -58,7 +58,7 @@ AP_DECLARE_DATA extern ap_filter_rec_t *
>> * @param c The current connection
>> * @return The new request_rec
>> */
>> -request_rec *ap_create_request(conn_rec *c);
>> +AP_DECLARE(request_rec *) ap_create_request(conn_rec *c);
>>
>> /**
>> * Read a request and fill in the fields.
>>
>>
>>
>
> Don't we need to do this in server/protocol.c as well?
>
> Regards
>
> Rüdiger
