Update Vulkan-Headers to 1.2.191; drop Video enum-variant edgecases #463
Conversation
MarijnS95
changed the title
|
The last commit is now replicating #352, without refactoring type handling in the generator. This is necessary because |
MarijnS95
changed the title
Vulkan 1.2.188 removed the only occurence of `ename:`, which was our heuristic to find a type that's a pointer to a static-sized array. It is now replaced with a "normal" `latexmath:` expression, but should still be dealt with appropriately. Note that all `latexmath:` conditions have been removed, as these fields should not be silently omitted. As of this Vulkan version only 3 exist, and all have their own edge-case.
This reverts commit 43dee26. Upstream is reverting back to inconsistent paths in [this commit] to counter unintentional header `#include` changes on the C-side: KhronosGroup/Vulkan-Docs#1573 [this commit]: KhronosGroup/Vulkan-Docs@8bde11c
Every `#include` path (except `"vk_platform.h"`, see previous commit) is relative to the `include/` directory or the current file, and does not look for a path relative to `include/vulkan/`.
MarijnS95
changed the title
I think that is fine. Can't expect stability guarantees for provisional extensions. Of course would be better to have it behind a toggle at one point. |
Yup, maybe need to have another crack at that some time soon; it's a known flaw in our API for now. |
Not creating a patch release for this because the changes are limited to the provisional video extension, and breaking changes at that. This should normally require a minor release as we haven't even been able to hide these behind a feature toggle yet :/
If the next updates/PRs are interesting, should we go and push a patch release anyway at risk of breaking someones experimental workflow for a bit?