On 12 Apr 2009, at 16:36, Kevac Marko wrote:

https://issues.apache.org/bugzilla/show_bug.cgi?id=46827

This patch adds ability to the mod_dbd Apache module to execute any
SQL statements after DB connection have been established.

Thanks for that.  I think it's looking good.

Couple of points:

1. You've exposed dbd_cfg_t in mod_dbd.h - why?

2.
+DBD_DECLARE_NONSTD(void) ap_dbd_sql_init(server_rec *s, const char *query)

That looks like an export for the API, but you haven't exposed it! Is that an
oversight, or am I missing something?

--
Nick Kew

Reply via email to