KhronosGroup / Vulkan-Samples

One stop solution for all Vulkan samples
Apache License 2.0
4.33k stars 648 forks source link

Unify the two classes vkb::core::VulkanResource and vkb::core::HPPVulkanResource into one: vkb::core::VulkanResource<BindingType, HandleType> #984

Closed asuessenbach closed 7 months ago

asuessenbach commented 8 months ago

Description

The next unification step of C- and C++-bindings: vkb::core[HPP]VulkanResource.

Build tested on Win10 with VS2022. Run tested on Win10 with NVidia GPU.

General Checklist:

Please ensure the following points are checked:

Sample Checklist

If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

gpx1000 commented 7 months ago

looks like a few merge conflicts.

Looks good to me other than that.