does anyone have any objections to this? all it is really doing is removing the MACRO definitions and splitting up the apr_dbm into 4 files (a interface, and 3 implementations) the interface has not changed at all.
The next part (registering/multiple DBMs) can be done after this stuff goes through.
..Ian
Ian Holsman wrote:
Changes since last time:
* NO CHANGE IN APR_DBM.H (so no diffs ;))
* added licence file to dbm_private and put it in
private/apr_dbm_private.h * GDBM is now there
* patch to apr_dbm.c.patch
* vtables are now static constants. (as per buckets)
* code style-cleanup to make it more apache. (might still be a stray
tab)
This version does NOT do: * apr_dbm_open(XXX,...) * apr_dbm_registration * allow for multiple DBMs.
all it really does at the moment is split the file into 3 parts.
All it requires for existing 'users' of the DBM is a re-link to the new library.
IF this patch is Ok, then I'll get the multi-dbm's working (which means changing the configure, and adding the registeration/open(XXX))
XXX_Usednames is not part of the hook, as you don't need a DBM open to call it. (and changing it to require it would break mod_dav, as it doesn't seem to have a open DBM when it calls the funciton)
??comments??? Ian
