KhronosGroup / Vulkan-Samples

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

Add missing checks in ~ImageView() and ~RenderPass(); adjust loading in multi_draw_indirect #1031

Closed asuessenbach closed 7 months ago

asuessenbach commented 7 months ago

Description

Re-add checks in ImageView and RenderPass destructors; Adjust scene loading sequence in sample multi_draw_indirect.

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

Fixes #1024

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:

gary-sweet commented 7 months ago

@marty-johnson59 We should consider merging this asap as it fixes a bunch of broken samples.

marty-johnson59 commented 7 months ago

OK, merging. LMK if issues.