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

2009-08-24 Thread Ruediger Pluem 


On 08/21/2009 09:40 PM, n...@apache.org wrote:
 Author: niq
 Date: Fri Aug 21 19:40:17 2009
 New Revision: 806692
 
 URL: http://svn.apache.org/viewvc?rev=806692view=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=806692r1=806691r2=806692view=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

2009-08-24 Thread Nick Kew 


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