Johannes Schneider
Johannes Schneider
I cherry-picked the commits which fixed the CI (didn't want to do a full rebase onto master already).
Ah nice: we can just offer `std::source_location::current())` as a default argument and it will tell us in the exception message where the error occured in the code.
[Vulkan Memory Allocator](https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator) version `3.0.1` is now available on conan! :partying_face: https://conan.io/center/vulkan-memory-allocator
Btw reading through Sascha Willem's code: Should we `inline` our code as well ?
Would it be possible to change the templates so they take the values of the structures as arguments? So instead of ```cpp auto descriptor_set_ai = wrapper::make_info(); descriptor_set_ai.descriptorPool = m_descriptor_pool; descriptor_set_ai.descriptorSetCount...
I will close this issue because we are using this in the codebase already together with [designated initializers](https://en.cppreference.com/w/cpp/language/aggregate_initialization#Designated_initializers). See [pull request 487](https://github.com/inexorgame/vulkan-renderer/pull/487) for example.
https://www.youtube.com/watch?v=ifmW3lOA6CA
https://www.youtube.com/watch?v=6NWfznwFnMs
https://www.youtube.com/watch?v=zSG6dPq57P8
Anoher thing I noticed is that on some systems there are special memory regions which we could use explicitely. I'm not sure in how far VMA does this by default,...