On Fri, Jan 22, 2016 at 09:32:31PM -0500, Russell Bryant wrote:
> Comparing None to an integer worked in Python 2, but fails in Python 3.
> 
> Signed-off-by: Russell Bryant <russ...@ovn.org>
> ---
>  python/ovs/reconnect.py | 2 +-
>  tests/test-reconnect.py | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/python/ovs/reconnect.py b/python/ovs/reconnect.py
> index 39dd556..d4f5951 100644
> --- a/python/ovs/reconnect.py
> +++ b/python/ovs/reconnect.py
> @@ -518,7 +518,7 @@ class Reconnect(object):
>          """Causes the next call to poller.block() to wake up when self.run()
>          should be called."""
>          timeout = self.timeout(now)
> -        if timeout >= 0:
> +        if timeout is not None and timeout >= 0:
>              poller.timer_wait(timeout)

I think that self.timeout() always either returns None or a nonnegative
value, so that you can just change this to:
        if timeout is not None:

Of course, the way you wrote it works too.

Acked-by: Ben Pfaff <b...@ovn.org>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to