Minor cleanup on swapchain creation#117
Conversation
cf1b755 to
ec63815
Compare
|
Any opinion on this PR? |
|
Sorry, yep this looks good to me. |
ddeb266 to
5e36441
Compare
|
This PR takes a number of iterations, as I still can't build on my own! I still have the problem, that Any idea on how to resolve that is very much appreciated! |
55f6017 to
558169e
Compare
558169e to
9e43862
Compare
SaschaWillems
left a comment
There was a problem hiding this comment.
Changes LGTM and work fine for me on Windows 11.
Yes, that works. Thanks for that hint. Now everything builds fine. Besides lots of warnings like: Besides that, is there a way to start all attachments one after the other and close them automatically? Similar to the batch mode in Vulkan-Samples? |
|
... and: starting with 09_shader_modules, there are three projects per chapter in the VS solution: $chapter, $chapter_shader, and $chapter_slang_shader. Is that intended? |
|
Yeah, that's sadly the way Visual Studio handles cmake. We can do things to organize it a bit; maybe can consolidate into one shader compile step, but that's just Visual Studio doing Visual Studio things. |

Essentially, introduced a little helper function to determine the minImageCount for the swapchain, and added some assertions.
As usual, if you think, this looks good, I can carry it over to the other chapters.
And still I'm not able to build or run this program, see #93.