-
Notifications
You must be signed in to change notification settings - Fork 82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vulkan.hpp support #68
Comments
This opens up a can of worms that I really don't intend to solve with vk-bootstrap. I think a single 'header' that inclues However, I don't know of which problem you speak of with 'extensions that need the dynamic loader'. I return the vk-bootstrap/src/VkBootstrap.h Line 217 in 95b82a9
So such code is possible with vk-bootstrap auto instance = vkb::InstanceBuilder().build().value(); //construct instance
VULKAN_HPP_DEFAULT_DISPATCHER.init(instance.vkGetInstanceProcAddr); Unless of course the above does not work, as I haven't tested it. (but I'm rather familiar with the internal guts of vulkan.hpp and vk-bootstrap that I don't know why this wouldn't work). The DynamicLoader of vulkan.hpp loads all function pointers, regardless of whether you tried to enable the appropriate function pointers. The |
I've found out that vulkan.hpp is planning on splitting the header into multiple files, which gave me the idea that there could be a 'vulkan.hpp vk-bootstra wrapper' header that more or less just takes the return type of the |
Are there still plans on implementing this enhancement in the future? |
Yes, unfortunately this repo is a low priority. Plus, work to make vulkan.hpp is a big undertaking and thus hard to just 'do'. I am very open to PR's though, if anyone is willing. |
Either via an external define or an alternative namespace, it would be great if the library allowed to return the Unique variants of Vulkan entities.
Trying to wrap the currently returned values in vulkan.hpp's Unique wrappers is easy in most cases, but when it comes to extensions that need a dynamic loader it becomes a mess (like the debug messenger). Ideally the "RAII variant" of the library would directly return the Unique wrappers to the user.
(Edit: I don't know how to flag as suggestion)
The text was updated successfully, but these errors were encountered: