anv: don't use Element.get in anv_entrypoints_gen.py
This has the potential to mask errors, since Element.get works like dict.get, returning None if the element isn't found. I think the reason that Element.get was used is that vulkan has one extension that isn't really an extension, and thus is missing the 'protect' field. This patch changes the behavior slightly by replacing get with explicit lookup in the Element.attrib dictionary, and using xpath to only iterate over extensions with a "protect" attribute. Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
This commit is contained in:
parent
4d4697f868
commit
9050138af7
|
@ -305,13 +305,14 @@ def get_entrypoints(doc, entrypoints_to_defines):
|
|||
def get_entrypoints_defines(doc):
|
||||
"""Maps entry points to extension defines."""
|
||||
entrypoints_to_defines = {}
|
||||
extensions = doc.findall('./extensions/extension')
|
||||
for extension in extensions:
|
||||
define = extension.get('protect')
|
||||
entrypoints = extension.findall('./require/command')
|
||||
for entrypoint in entrypoints:
|
||||
fullname = entrypoint.get('name')
|
||||
|
||||
for extension in doc.findall('./extensions/extension[@protect]'):
|
||||
define = extension.attrib['protect']
|
||||
|
||||
for entrypoint in extension.findall('./require/command'):
|
||||
fullname = entrypoint.attrib['name']
|
||||
entrypoints_to_defines[fullname] = define
|
||||
|
||||
return entrypoints_to_defines
|
||||
|
||||
|
||||
|
|
Loading…
Reference in New Issue