[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: commit: ldap/libraries/libldap result.c



hyc@OpenLDAP.org wrote:
> Update of /repo/OpenLDAP/pkg/ldap/libraries/libldap
> 
> Modified Files:
> 	result.c  1.149 -> 1.150
> 
> Log Message:
> Fairly sure this is what the TIMEOUT option should always have been for

Not strictly related, but does it make sense to have a malloc for a
struct timeval in global/per handler data?  Wouldn't it be any better to
use tv_sec = -1 to indicate no timeout?  We should be able to change
that, since the struct is opaque...

p.



Ing. Pierangelo Masarati
OpenLDAP Core Team

SysNet s.n.c.
Via Dossi, 8 - 27100 Pavia - ITALIA
http://www.sys-net.it
------------------------------------------
Office:   +39.02.23998309
Mobile:   +39.333.4963172
Email:    pierangelo.masarati@sys-net.it
------------------------------------------