Re: cvs: php4 /ext/ldap ldap.c

On Wed, Jun 26, 2002 at 12:13:50PM +0200, derick@php.net wrote: > This should work, all *_s functions (in this patch) return the LDAP error > code, from RFC 1328 (http://asg.web.cmu.edu/rfc/rfc1823.html): > > ldap_add_s() is synchronous, returning the LDAP error code indicating the > outcome of the operation. > > ldap_add() is asynchronous, returning the message id of the operation it > initiates, or -1 in case of trouble. > > And from the url you specified: > > All synchronous rou- > tines return an indication of the outcome of the operation (e.g, the > constant LDAP_SUCCESS or some other result code). > > > So, I think it's all ok :) Sorry, I was too quick. I remembered that bind.. returned -1 on error, but right, not the synchronous version. Another thing that should be done, it's not a very high priority for me, is to change to zend_parse_parameters() all over. I have plans to do this slowly over time (in between other things), but if someone has the time... Stig