Add require_results attribute to the Vulkan procedures that return vulkan.Result #6307
SolidAlloy
started this conversation in
Proposals
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
A lot of Vulkan procedures return a
vulkan.Resultvalue that needs to be checked. It is commonly implemented with avk_check()procedure. However, there are a number of void procedures. Thus, it is easy to miss checking the return value of a Vulkan call because not every function needs it, and if you forget, it doesn't pop up as an error.The proposal is to tweak the bindings generator to add the
require_resultsattribute to any Vulkan function that returnsvulkan.Result.Beta Was this translation helpful? Give feedback.
All reactions