Looks good to me. It'd be nice if we could add some unit tests for this, though I'm not sure if it's worth it.
Ethan On Mon, Jul 2, 2012 at 10:35 AM, Ben Pfaff <b...@nicira.com> wrote: > Bug #12301. > Reported-by: Mike Kruze <mkr...@nicira.com> > Signed-off-by: Ben Pfaff <b...@nicira.com> > --- > python/ovs/jsonrpc.py | 2 +- > python/ovs/stream.py | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/python/ovs/jsonrpc.py b/python/ovs/jsonrpc.py > index cf08131..a054401 100644 > --- a/python/ovs/jsonrpc.py > +++ b/python/ovs/jsonrpc.py > @@ -210,7 +210,7 @@ class Connection(object): > if not self.status: > self.stream.run_wait(poller) > if len(self.output): > - self.stream.send_wait() > + self.stream.send_wait(poller) > > def get_status(self): > return self.status > diff --git a/python/ovs/stream.py b/python/ovs/stream.py > index 82ea0c1..aa512f6 100644 > --- a/python/ovs/stream.py > +++ b/python/ovs/stream.py > @@ -113,7 +113,7 @@ class Stream(object): > break > stream.run() > poller = ovs.poller.Poller() > - stream.run_wait() > + stream.run_wait(poller) > stream.connect_wait(poller) > poller.block() > assert error != errno.EINPROGRESS > -- > 1.7.2.5 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
