Rainer Gerhards writes:
 > Markus,
 > 
 > I had a somewhat deeper look at dbi_main now. I think four or five
 > functions would need to be changed in the way you propose. However, as

The functions dbi_initialize(), dbi_shutdown(),
dbi_driver_list(), and dbi_driver_open() access rootdriver. All of
these can easily be modified to accept the sentinel as an additional
parameter.

 > this is an interface change, the previous functions would need to remain
 > as is (including the plugin problem). While I looked at the code, I

This is how we handled most incompatible changes in the past. The old
functions were usually labelled as "deprecated", but in fact most of
them are still around. We'll just have to find suitable names for the
additional functions.

 > think a few mutex lock/unlock calls would also make it thread-safe. That
 > is not a big issue for me, as rsyslog allows plugins to run on a single
 > thread, but it may be a benefit for other situations.
 > 
 > If you like, I can try to apply a number of changes and provide a patch
 > file for your review.
 > 

Patches are more than welcome. Feel free to send them at your earliest
convenience.

regards,
Markus

-- 
Markus Hoenicka
[EMAIL PROTECTED]
(Spam-protected email: replace the quadrupeds with "mhoenicka")
http://www.mhoenicka.de


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
libdbi-users mailing list
libdbi-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libdbi-users

Reply via email to