ovs.db.types: Simplify code to avoid try/except case.
authorBen Pfaff <blp@nicira.com>
Tue, 23 Aug 2011 16:58:49 +0000 (09:58 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 24 Aug 2011 18:57:43 +0000 (11:57 -0700)
Also fixes a typo that caused one version of the error message to have a
hyphen and the other to have a space.

Suggested-by: Reid Price <reid@nicira.com>
python/ovs/db/types.py

index f7bc2482b1c5407707a72060d5007b2c6a89b670..416d7456baa52142fb79f5a9010027f4dd4c0ae4 100644 (file)
@@ -30,16 +30,14 @@ class AtomicType(object):
             for atomic_type in ATOMIC_TYPES:
                 if s == atomic_type.name:
                     return atomic_type
-        raise error.Error('"%s" is not an atomic type' % s)
+        raise error.Error('"%s" is not an atomic-type' % s, s)
 
     @staticmethod
     def from_json(json):
         if type(json) not in [str, unicode]:
             raise error.Error("atomic-type expected", json)
-        try:
+        else:
             return AtomicType.from_string(json)
-        except error.Error:
-            raise error.Error('"%s" is not an atomic-type' % json, json)
 
     def __str__(self):
         return self.name