Re: cvs commit: apache-1.3/src/include http_protocol.h

["William A. Rowe, Jr." <wr...@covalent.net>]

> wrowe       02/01/21 12:29:12
> 
>   Modified:    src/include http_protocol.h
>   Log:
>     ap_rprintf is implemented as API_EXPORT() in http_protocol.c.
>     Therefore, make the prototype and implementation consistant.
>   
>     The question remains, why no compiler warning/emit?  Because MSVC [my
>     version, at least] must have changed all of the ap_fn(foo, ...) decl
>     from _stdcall to _cdecl, on it's own!
>   
>     I suggest our declarations are still borked, but they have worked only
>     because MSVC ignored our poor judgement :)


I just proved this theory to myself by reapplying the .c patches without
reapplying the .h patches to make these fns _NONSTD.  No warning.  Changing
the return type for one of the ap_rprintf() declarations caused a prototype
mismatch warning - so we know the http_protocol.c really is reading the .h
prototypes.

I could commit the correct fixes or leave them alone.  I don't care at this
point.

Bill