Re: result() timeout

[Michael <mi...@st...>]

David Leonard wrote:
> 
> currently when l.result(msgid, all, timeout) is called with a nonzero timeout
> and timeout expires, then None is returned rather than the usual
>  (res_type, value) tuple.
> 
> this change makes result() raise an ldap.TIMEOUT exception instead.
> supposedly this makes it easier to write code to handle result timeouts?

I'm fine with either way. But ldap.TIMEOUT should only be raised if
the timeout value was really nonzero. If the timeout value is zero
returning solely None is the most reasonable way.

I have to look into ldapthreading if I have to change something.

Ciao, Michael.