On Fri, Apr 14, 2017 at 01:14:08PM +0000, hyc@symas.com wrote:

okuznik@symas.com wrote:

...

Integrating with libevent, I would find it useful if libldap_r provided a recursive mutex as well, since this is what libevent uses.

You mean libldap_r/rmutex.c ?

Hmm, I see that now, but is quite heavy weight if this already exists in many implementations natively, often at no cost at all. What if I renamed the current ldap_pvt_*_rmutex_* functions to ldap_int_* and deferred to them in the two implementations that can't do recursive mutexes only?

This would be an ABI break for rmutex users, other than that, there seems to be no expectation that two versions of libldap_r compiled against a different thread implementation should be ABI compatible already.