jpbruyere / vk.net

Vulkan binding for .net
MIT License
32 stars 3 forks source link

Added vkEnumerateInstanceVersion to preloaded commands #11

Closed TechLiam closed 4 years ago

TechLiam commented 4 years ago

vkEnumerateInstanceVersion might be wanted when creating an instance of validating the version of Vulkan on a system before starting a program

jpbruyere commented 4 years ago

I was also thinking that It should be possible to automate the building of the preloaded list by checking the arguments of the functions (if no vkInstance or vkDevice...).

TechLiam commented 4 years ago

That a good idea if I come across anything else like this and you have not done this change I will look at doing that

jpbruyere commented 4 years ago

I've made it in a new branch ('AutomatedPreloadedCmdList'), there is no other preload cmd than those already manually added, so maybe it's a bit unnecessary to autogen this. In the case new commands appears in future version of vulkan that need to be preloaded, it would be transparent, but autogen is always exposed to special case that may failed. I'll think a bit about this before pushing or not to master.