[ovs-dev] [PATCH v2 0/2] python/ovs: add tcp socket support
Ben Pfaff
blp at nicira.com
Tue Oct 16 17:20:30 UTC 2012
On Tue, Oct 16, 2012 at 06:57:27AM +0900, Isaku Yamahata wrote:
> On Mon, Oct 15, 2012 at 12:10:16PM -0700, Reid Price wrote:
> > I see that Ben already reviewed and pushed, only saw a comment-typo
> > and a message-typo that Ben seems to have seen. My only nit would be
> > using some generic Error for the parameter checking (ValueError, or
> > ovs-specific) instead of raising Exception. The register decorators /
> > helper classes seem reasonable and clean, if a little overkill
> > compared to a conditional in open(). Thanks!
>
> Thank you for review. Here's the patch to replace Exception with ValueError.
Thanks, I applied this to master.
More information about the dev
mailing list