You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Ruediger Pluem <rp...@apache.org> on 2009/08/22 07:00:09 UTC

Re: svn commit: r806692 - in /httpd/httpd/trunk/modules/database: mod_dbd.c mod_dbd.h


On 08/21/2009 09:40 PM, niq@apache.org wrote:
> Author: niq
> Date: Fri Aug 21 19:40:17 2009
> New Revision: 806692
> 
> URL: http://svn.apache.org/viewvc?rev=806692&view=rev
> Log:
> Switch mod_dbd to using APR_STATUS_IS_foo macros in error handling
> 
> Modified:
>     httpd/httpd/trunk/modules/database/mod_dbd.c
>     httpd/httpd/trunk/modules/database/mod_dbd.h
> 

> 
> Modified: httpd/httpd/trunk/modules/database/mod_dbd.h
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/database/mod_dbd.h?rev=806692&r1=806691&r2=806692&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/database/mod_dbd.h (original)
> +++ httpd/httpd/trunk/modules/database/mod_dbd.h Fri Aug 21 19:40:17 2009
> @@ -116,7 +116,7 @@
>  APR_DECLARE_OPTIONAL_FN(void, ap_dbd_prepare, (server_rec*, const char*, const char*));
>  
>  APR_DECLARE_EXTERNAL_HOOK(dbd, DBD, apr_status_t, post_connect,
> -                          (apr_pool_t *, dbd_cfg_t *, ap_dbd_t *));
> +                          (apr_pool_t *, dbd_cfg_t *, ap_dbd_t *))

Why this one?

Regards

RĂ¼diger

Re: svn commit: r806692 - in /httpd/httpd/trunk/modules/database: mod_dbd.c mod_dbd.h

Posted by Nick Kew <ni...@webthing.com>.
On 22 Aug 2009, at 06:00, Ruediger Pluem wrote:

>>  APR_DECLARE_EXTERNAL_HOOK(dbd, DBD, apr_status_t, post_connect,
>> -                          (apr_pool_t *, dbd_cfg_t *, ap_dbd_t *));
>> +                          (apr_pool_t *, dbd_cfg_t *, ap_dbd_t *))
>
> Why this one?

Oh, er, just shutting up a compiler warning about an empty declaration.
Didn't seem as significant as the real change in the .c.

-- 
Nick Kew