[PATCH] python/ovs/socket_util: don't use Exception, but ValueError

Isaku Yamahata yamahata at valinux.co.jp
Mon Oct 15 21:46:07 UTC 2012


67656b9ff297f305b3bfcca2868e8e870e108283
used Exception, but it should be more specific error as Reid Pointed out.
Use ValueError instread of Exception.

Signed-off-by: Isaku Yamahata <yamahata at valinux.co.jp>
---
 python/ovs/socket_util.py |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/python/ovs/socket_util.py b/python/ovs/socket_util.py
index 845511e..dd45fe4 100644
--- a/python/ovs/socket_util.py
+++ b/python/ovs/socket_util.py
@@ -88,13 +88,13 @@ def inet_parse_active(target, default_port):
     address = target.split(":")
     host_name = address[0]
     if not host_name:
-        raise Exception("%s: bad peer name format" % target)
+        raise ValueError("%s: bad peer name format" % target)
     if len(address) >= 2:
         port = int(address[1])
     elif default_port:
         port = default_port
     else:
-        raise Exception("%s: port number must be specified" % target)
+        raise ValueError("%s: port number must be specified" % target)
     return (host_name, port)
 
 
@@ -187,6 +187,6 @@ def set_nonblocking(sock):
 
 def set_dscp(sock, dscp):
     if dscp > 63:
-        raise Exception("Invalid dscp %d" % dscp)
+        raise ValueError("Invalid dscp %d" % dscp)
     val = dscp << 2
     sock.setsockopt(socket.IPPROTO_IP, socket.IP_TOS, val)
-- 
1.7.10.4




-- 
yamahata


More information about the dev mailing list