XML <extension>: Replace remaining condition with depends#1088
XML <extension>: Replace remaining condition with depends#1088SunSerega wants to merge 3 commits intoKhronosGroup:mainfrom
<extension>: Replace remaining condition with depends#1088Conversation
|
The "conditions" are currently used by the extension header generation scripts: I'm OK doing something like this PR to remove redundancy, but we'll need to update the header generation scripts as well if we make this change. |
|
Ah, I was somehow sure there were more |
|
The 'condition' is something OpenCL-specific. The 'depends' was introduced when syncing with the Vulkan scripts and has the same restrictions, it only speaks to extension and core version dependencies. |
Well, it can represent more cases. For instance in this case: Here, I don't know how to convert this But what about all the extensions with |
Asked at the end of #950. This in particular wasn't answered yet, but after getting a bit more comfortable with new XML - I'm pretty sure this does need to be fixed.
Note: There are also a bunch of
<require>tags withconditioninstead of thedependsattribute.The
dependsattribute is being used for these too, but only in one place:But half of the
conditionattributes here use negation:And there is also one like this:
Which is neither a version nor an extension name, so I guess this one cannot be converted to
depends?