Hans Leidekker hans@it.vu.nl writes:
Yes, strictly speaking we should check all 4 reference handling functions that are absent from your libldap_r, but I'd rather keep the configure checks to an absolute minimum. Since these functions are related I doubt that there are versions of libldap_r around where less than 4 of them are missing. And when that happens we can deal with it ;-)
It's really not much more work to do it right the first time around, and it makes the code much clearer. Otherwise it looks quite strange to have #ifdefs for a function you aren't even calling.