Johannes Schneider

Results 156 comments of 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.

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,...