httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Eissing <stefan.eiss...@greenbytes.de>
Subject Re: svn commit: r1748155 - /httpd/httpd/trunk/modules/dav/main/mod_dav.h
Date Mon, 13 Jun 2016 09:53:24 GMT
Hmm, there is a "typedef struct dav_response dav_response" further and the new typedef of the
same struct gave an error in clang. I thought this was the best way to fix this...

-Stefan

> Am 13.06.2016 um 11:42 schrieb Ruediger Pluem <rpluem@apache.org>:
> 
> 
> 
> On 06/13/2016 11:31 AM, icing@apache.org wrote:
>> Author: icing
>> Date: Mon Jun 13 09:31:09 2016
>> New Revision: 1748155
>> 
>> URL: http://svn.apache.org/viewvc?rev=1748155&view=rev
>> Log:
>> mod_dav: making mod_dav.h compile with clang again
>> 
>> Modified:
>>    httpd/httpd/trunk/modules/dav/main/mod_dav.h
>> 
>> Modified: httpd/httpd/trunk/modules/dav/main/mod_dav.h
>> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/dav/main/mod_dav.h?rev=1748155&r1=1748154&r2=1748155&view=diff
>> ==============================================================================
>> --- httpd/httpd/trunk/modules/dav/main/mod_dav.h (original)
>> +++ httpd/httpd/trunk/modules/dav/main/mod_dav.h Mon Jun 13 09:31:09 2016
>> @@ -486,7 +486,7 @@ typedef struct
>> } dav_get_props_result;
>> 
>> /* holds the contents of a <response> element */
>> -typedef struct dav_response
>> +struct dav_response
> 
> Shouldn't that be
> 
> typedef struct
> 
>> {
>>     const char *href;           /* always */
>>     const char *desc;           /* optional description at <response> level
*/
>> @@ -497,7 +497,7 @@ typedef struct dav_response
>>     int status;
>> 
>>     struct dav_response *next;
>> -} dav_response;
> 
> and this line kept
> 
> Regards
> 
> RĂ¼diger


Mime
View raw message