Before this change, enums that have one member were formatted as, e.g.:
"one of xyzzy, , or "
This changes them to be formatted as:
"must be xyzzy"
which makes much more sense.
(An enum with one member may make some sense if you are trying to leave
the possibility for future expansion.)
Signed-off-by: Ben Pfaff <[email protected]>
---
python/ovs/db/types.py | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/python/ovs/db/types.py b/python/ovs/db/types.py
index 5865acd..bd1c259 100644
--- a/python/ovs/db/types.py
+++ b/python/ovs/db/types.py
@@ -287,7 +287,9 @@ class BaseType(object):
if self.enum:
literals = [value.toEnglish(escapeLiteral)
for value in self.enum.values]
- if len(literals) == 2:
+ if len(literals) == 1:
+ english = 'must be %s' % (literals[0])
+ elif len(literals) == 2:
english = 'either %s or %s' % (literals[0], literals[1])
else:
english = 'one of %s, %s, or %s' % (literals[0],
--
1.7.2.5
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev