I was implementing the same enum support in broadcom's gen_pack_header.py,
and did this same simplification there.
---
src/intel/genxml/gen_pack_header.py | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/intel/genxml/gen_pack_header.py
b/src/intel/genxml/gen_pack_header.py
index 9021f008a823..405f600174d2 100644
--- a/src/intel/genxml/gen_pack_header.py
+++ b/src/intel/genxml/gen_pack_header.py
@@ -493,7 +493,8 @@ class Parser(object):
self.instruction = None
self.structs = {}
- self.enums = {}
+ # Set of enum names we've seen.
+ self.enums = set()
self.registers = {}
def gen_prefix(self, name):
@@ -540,7 +541,7 @@ class Parser(object):
elif name == "enum":
self.values = []
self.enum = safe_name(attrs["name"])
- self.enums[attrs["name"]] = 1
+ self.enums.add(attrs["name"])
if "prefix" in attrs:
self.prefix = safe_name(attrs["prefix"])
else:
--
2.14.1
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
