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 <[email protected]> wrote: > Bug #12301. > Reported-by: Mike Kruze <[email protected]> > Signed-off-by: Ben Pfaff <[email protected]> > --- > 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 > [email protected] > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
