Re: Bug in LDAPObject.c, l_ldap_result()

[David L. <dav...@it...>]

On Sun, 30 Jun 2002, Michael Str=F6der typed thusly:

> > | >I also think the C function needs
> > | >to be modified to treat Py_None as NULL to allow python code to
> > | >specify a timeout that is indefinitely long.
> > |
> > | IMHO that's done with timeout=3D-1. Looking at l_ldap_result() in
> > | LDAPObject.c it seems to be implemented correctly
> >
> > Ahh, yeah, that is one suitable convention.  Doesn't
> >     timeout=3DNone
> > sound amusingly accurate though?  (in english at least)
>
> Well, I have no clue why David chose -1 instead of None.

that's the C/unix convention.

however, looking at the documentation of select.select() it seems that
using None could be better justified

  select(rlist, wlist, xlist[, timeout]) -> (rlist, wlist, xlist)
  [...]
  The optional 4th argument specifies a timeout in seconds; it may be
  a floating point number to specify fractions of seconds.  If it is absent
  or None, the call will never time out.

d
--=20
David Leonard                           Dav...@it...
Dept of Inf. Tech. and Elec. Engg   _   Ph:+61 404 844 850
The University of Queensland       |+|  http://www.itee.uq.edu.au/~leonard/
QLD 4072  AUSTRALIA               ~` '~ B73CD65FBEF4C089B79A8EBADF1A932F13E=
A0FC8