Handle window resizing about vookoo HOT 7 CLOSED

I have taken the approach of just recreating the minimum within the vku::Window.
Everything is working but I am getting a few validation layer messages.

Another issue I know can (and given Murphy's law, I'm sure it will on somebody's system) is that I trigger the resize change by catching the exception thrown by the presentQueue method. I think it is not strictly required that an implementation throw this exception.
Every time the window is resized I recreate:

• the swap chain
• the images
• the depth stencil
• the frame buffers.

As for the user's program, it is expected to recreate any affected pipelines.

For those who want to check it out, the current state is at https://github.com/FunMiles/Vookoo/tree/resize

For the moment, only the teapot example does the correct pipeline re-creation.
Edit: All demos work without crashing. The first 3 are not recreating a pipeline with the right size yet.

from vookoo.

There is a slight difficulty with the static commands. I've fixed the issue for the texture example.
The static drawing build function (functor passed to setStaticCommands) needs to rebuild the pipeline and command buffers any time the window size is changed. I've made a copy of the std::function being passed. But of course all the variables in the functions need to stay in scope. With a capturing by reference lambda, this can yield to some potential errors and crashes.

from vookoo.

I'm please to see that this is working much better now. It is very hard to diagnose some of these issues
as implementations interpret the standard in different ways.

from vookoo.

Probably fixed by @FunMiles

from vookoo.

Hi guys. I've been very busy with work and then after I had some free time, my linux box didn't want to run Vulkan anymore.
Today I got everything back and everything works on linux as well.
I did hit a little issue for which I thought I had submitted a PR. I guess I must have inflated my similar PR to imgui to think I also did it for Vookoo.
The Validation layer is no more Lunarg but Khronos.
If nobody else has an issue with that, I'll submit a PR. MoltenVK has dropped the Lunarg validation layer, so the code won't run on MacOS.

from vookoo.

Thanks for the confirmation!
And sure, go ahead with your PRs.

from vookoo.

PR submitted.

from vookoo.