Skip to content

Commit 558169e

Browse files
asuessenbachasuessenbach
committed
Carried changes over to all chapters.
1 parent 399bade commit 558169e

30 files changed

+1113
-854
lines changed

‎attachments/06_swap_chain_creation.cpp‎

Lines changed: 18 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ class HelloTriangleApplication {
5252
vk::raii::Queue queue = nullptr;
5353
vk::raii::SwapchainKHR swapChain = nullptr;
5454
std::vector<vk::Image> swapChainImages;
55-
vk::Format swapChainImageFormat = vk::Format::eUndefined;
55+
vk::SurfaceFormatKHR swapChainSurfaceFormat;
5656
vk::Extent2D swapChainExtent;
5757
std::vector<vk::raii::ImageView> swapChainImageViews;
5858

@@ -245,19 +245,21 @@ class HelloTriangleApplication {
245245
}
246246

247247
void createSwapChain() {
248-
auto surfaceCapabilities = physicalDevice.getSurfaceCapabilitiesKHR( surface );
248+
auto surfaceCapabilities = physicalDevice.getSurfaceCapabilitiesKHR( surface );
249+
swapChainExtent = chooseSwapExtent( surfaceCapabilities );
250+
swapChainSurfaceFormat = chooseSwapSurfaceFormat( physicalDevice.getSurfaceFormatsKHR( surface ) );
249251
vk::SwapchainCreateInfoKHR swapChainCreateInfo{ .surface = surface,
250-
.minImageCount = chooseSwapMinImageCount( surfaceCapabilities ),
251-
.imageFormat = chooseSwapSurfaceFormat( physicalDevice.getSurfaceFormatsKHR( surface ) ),
252-
.imageColorSpace = vk::ColorSpaceKHR::eSrgbNonlinear,
253-
.imageExtent = chooseSwapExtent( surfaceCapabilities ),
254-
.imageArrayLayers = 1,
255-
.imageUsage = vk::ImageUsageFlagBits::eColorAttachment,
256-
.imageSharingMode = vk::SharingMode::eExclusive,
257-
.preTransform = surfaceCapabilities.currentTransform,
258-
.compositeAlpha = vk::CompositeAlphaFlagBitsKHR::eOpaque,
259-
.presentMode = chooseSwapPresentMode( physicalDevice.getSurfacePresentModesKHR( surface ) ),
260-
.clipped = true };
252+
.minImageCount = chooseSwapMinImageCount( surfaceCapabilities ),
253+
.imageFormat = swapChainSurfaceFormat.format,
254+
.imageColorSpace = swapChainSurfaceFormat.colorSpace,
255+
.imageExtent = swapChainExtent,
256+
.imageArrayLayers = 1,
257+
.imageUsage = vk::ImageUsageFlagBits::eColorAttachment,
258+
.imageSharingMode = vk::SharingMode::eExclusive,
259+
.preTransform = surfaceCapabilities.currentTransform,
260+
.compositeAlpha = vk::CompositeAlphaFlagBitsKHR::eOpaque,
261+
.presentMode = chooseSwapPresentMode( physicalDevice.getSurfacePresentModesKHR( surface ) ),
262+
.clipped = true };
261263

262264
swapChain = vk::raii::SwapchainKHR( device, swapChainCreateInfo );
263265
swapChainImages = swapChain.getImages();
@@ -271,16 +273,16 @@ class HelloTriangleApplication {
271273
return minImageCount;
272274
}
273275

274-
static vk::Format chooseSwapSurfaceFormat(std::vector<vk::SurfaceFormatKHR> const & availableFormats) {
276+
static vk::SurfaceFormatKHR chooseSwapSurfaceFormat(std::vector<vk::SurfaceFormatKHR> const & availableFormats) {
275277
assert(!availableFormats.empty());
276278
const auto formatIt = std::ranges::find_if(
277279
availableFormats,
278280
[]( const auto & format ) { return format.format == vk::Format::eB8G8R8A8Srgb && format.colorSpace == vk::ColorSpaceKHR::eSrgbNonlinear; } );
279-
return formatIt != availableFormats.end() ? formatIt->format : availableFormats[0].format;
281+
return formatIt != availableFormats.end() ? *formatIt : availableFormats[0];
280282
}
281283

282284
static vk::PresentModeKHR chooseSwapPresentMode(const std::vector<vk::PresentModeKHR>& availablePresentModes) {
283-
assert(std::ranges::any_of(availablePresentModes, [](auto presentMode){ return presentMode == vk::PresentMode::eFifo; }));
285+
assert(std::ranges::any_of(availablePresentModes, [](auto presentMode){ return presentMode == vk::PresentModeKHR::eFifo; }));
284286
return std::ranges::any_of(availablePresentModes,
285287
[](const vk::PresentModeKHR value) { return vk::PresentModeKHR::eMailbox == value; } ) ? vk::PresentModeKHR::eMailbox : vk::PresentModeKHR::eFifo;
286288
}

‎attachments/07_image_views.cpp‎

Lines changed: 36 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
#include <memory>
77
#include <algorithm>
88
#include <limits>
9+
#include <assert.h>
910

1011
#ifdef __INTELLISENSE__
1112
#include <vulkan/vulkan_raii.hpp>
@@ -49,11 +50,10 @@ class HelloTriangleApplication {
4950
vk::raii::PhysicalDevice physicalDevice = nullptr;
5051
vk::raii::Device device = nullptr;
5152
vk::raii::Queue queue = nullptr;
52-
53-
vk::raii::SwapchainKHR swapChain = nullptr;
54-
std::vector<vk::Image> swapChainImages;
55-
vk::Format swapChainImageFormat = vk::Format::eUndefined;
56-
vk::Extent2D swapChainExtent;
53+
vk::raii::SwapchainKHR swapChain = nullptr;
54+
std::vector<vk::Image> swapChainImages;
55+
vk::SurfaceFormatKHR swapChainSurfaceFormat;
56+
vk::Extent2D swapChainExtent;
5757
std::vector<vk::raii::ImageView> swapChainImageViews;
5858

5959
std::vector<const char*> requiredDeviceExtension = {
@@ -247,18 +247,20 @@ class HelloTriangleApplication {
247247

248248
void createSwapChain() {
249249
auto surfaceCapabilities = physicalDevice.getSurfaceCapabilitiesKHR( surface );
250-
swapChainImageFormat = chooseSwapSurfaceFormat(physicalDevice.getSurfaceFormatsKHR( surface ));
251-
swapChainExtent = chooseSwapExtent(surfaceCapabilities);
252-
auto minImageCount = std::max( 3u, surfaceCapabilities.minImageCount );
253-
minImageCount = ( surfaceCapabilities.maxImageCount > 0 && minImageCount > surfaceCapabilities.maxImageCount ) ? surfaceCapabilities.maxImageCount : minImageCount;
254-
vk::SwapchainCreateInfoKHR swapChainCreateInfo{
255-
.surface = surface, .minImageCount = minImageCount,
256-
.imageFormat = swapChainImageFormat, .imageColorSpace = vk::ColorSpaceKHR::eSrgbNonlinear,
257-
.imageExtent = swapChainExtent, .imageArrayLayers =1,
258-
.imageUsage = vk::ImageUsageFlagBits::eColorAttachment, .imageSharingMode = vk::SharingMode::eExclusive,
259-
.preTransform = surfaceCapabilities.currentTransform, .compositeAlpha = vk::CompositeAlphaFlagBitsKHR::eOpaque,
260-
.presentMode = chooseSwapPresentMode(physicalDevice.getSurfacePresentModesKHR( surface )),
261-
.clipped = true };
250+
swapChainExtent = chooseSwapExtent( surfaceCapabilities );
251+
swapChainSurfaceFormat = chooseSwapSurfaceFormat( physicalDevice.getSurfaceFormatsKHR( surface ) );
252+
vk::SwapchainCreateInfoKHR swapChainCreateInfo{ .surface = surface,
253+
.minImageCount = chooseSwapMinImageCount( surfaceCapabilities ),
254+
.imageFormat = swapChainSurfaceFormat.format,
255+
.imageColorSpace = swapChainSurfaceFormat.colorSpace,
256+
.imageExtent = swapChainExtent,
257+
.imageArrayLayers = 1,
258+
.imageUsage = vk::ImageUsageFlagBits::eColorAttachment,
259+
.imageSharingMode = vk::SharingMode::eExclusive,
260+
.preTransform = surfaceCapabilities.currentTransform,
261+
.compositeAlpha = vk::CompositeAlphaFlagBitsKHR::eOpaque,
262+
.presentMode = chooseSwapPresentMode( physicalDevice.getSurfacePresentModesKHR( surface ) ),
263+
.clipped = true };
262264

263265
swapChain = vk::raii::SwapchainKHR( device, swapChainCreateInfo );
264266
swapChainImages = swapChain.getImages();
@@ -267,7 +269,7 @@ class HelloTriangleApplication {
267269
void createImageViews() {
268270
swapChainImageViews.clear();
269271

270-
vk::ImageViewCreateInfo imageViewCreateInfo{ .viewType = vk::ImageViewType::e2D, .format = swapChainImageFormat,
272+
vk::ImageViewCreateInfo imageViewCreateInfo{ .viewType = vk::ImageViewType::e2D, .format = swapChainSurfaceFormat.format,
271273
.subresourceRange = { vk::ImageAspectFlagBits::eColor, 0, 1, 0, 1 } };
272274
for ( auto image : swapChainImages )
273275
{
@@ -276,22 +278,30 @@ class HelloTriangleApplication {
276278
}
277279
}
278280

279-
static vk::Format chooseSwapSurfaceFormat(const std::vector<vk::SurfaceFormatKHR>& availableFormats) {
280-
const auto formatIt = std::ranges::find_if(availableFormats,
281-
[](const auto& format) {
282-
return format.format == vk::Format::eB8G8R8A8Srgb &&
283-
format.colorSpace == vk::ColorSpaceKHR::eSrgbNonlinear;
284-
});
285-
return formatIt != availableFormats.end() ? formatIt->format : availableFormats[0].format;
281+
static uint32_t chooseSwapMinImageCount(vk::SurfaceCapabilitiesKHR const & surfaceCapabilities) {
282+
auto minImageCount = std::max( 3u, surfaceCapabilities.minImageCount );
283+
if ((0 < surfaceCapabilities.maxImageCount) && (surfaceCapabilities.maxImageCount < minImageCount)) {
284+
minImageCount = surfaceCapabilities.maxImageCount;
285+
}
286+
return minImageCount;
287+
}
288+
289+
static vk::SurfaceFormatKHR chooseSwapSurfaceFormat(const std::vector<vk::SurfaceFormatKHR>& availableFormats) {
290+
assert(!availableFormats.empty());
291+
const auto formatIt = std::ranges::find_if(
292+
availableFormats,
293+
[]( const auto & format ) { return format.format == vk::Format::eB8G8R8A8Srgb && format.colorSpace == vk::ColorSpaceKHR::eSrgbNonlinear; } );
294+
return formatIt != availableFormats.end() ? *formatIt : availableFormats[0];
286295
}
287296

288297
static vk::PresentModeKHR chooseSwapPresentMode(const std::vector<vk::PresentModeKHR>& availablePresentModes) {
298+
assert(std::ranges::any_of(availablePresentModes, [](auto presentMode){ return presentMode == vk::PresentModeKHR::eFifo; }));
289299
return std::ranges::any_of(availablePresentModes,
290300
[](const vk::PresentModeKHR value) { return vk::PresentModeKHR::eMailbox == value; } ) ? vk::PresentModeKHR::eMailbox : vk::PresentModeKHR::eFifo;
291301
}
292302

293303
vk::Extent2D chooseSwapExtent(const vk::SurfaceCapabilitiesKHR& capabilities) {
294-
if (capabilities.currentExtent.width != std::numeric_limits<uint32_t>::max()) {
304+
if (capabilities.currentExtent.width != 0xFFFFFFFF) {
295305
return capabilities.currentExtent;
296306
}
297307
int width, height;

‎attachments/08_graphics_pipeline.cpp‎

Lines changed: 36 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
#include <memory>
77
#include <algorithm>
88
#include <limits>
9+
#include <assert.h>
910

1011
#ifdef __INTELLISENSE__
1112
#include <vulkan/vulkan_raii.hpp>
@@ -49,11 +50,10 @@ class HelloTriangleApplication {
4950
vk::raii::PhysicalDevice physicalDevice = nullptr;
5051
vk::raii::Device device = nullptr;
5152
vk::raii::Queue queue = nullptr;
52-
53-
vk::raii::SwapchainKHR swapChain = nullptr;
54-
std::vector<vk::Image> swapChainImages;
55-
vk::Format swapChainImageFormat = vk::Format::eUndefined;
56-
vk::Extent2D swapChainExtent;
53+
vk::raii::SwapchainKHR swapChain = nullptr;
54+
std::vector<vk::Image> swapChainImages;
55+
vk::SurfaceFormatKHR swapChainSurfaceFormat;
56+
vk::Extent2D swapChainExtent;
5757
std::vector<vk::raii::ImageView> swapChainImageViews;
5858

5959
std::vector<const char*> requiredDeviceExtension = {
@@ -248,18 +248,20 @@ class HelloTriangleApplication {
248248

249249
void createSwapChain() {
250250
auto surfaceCapabilities = physicalDevice.getSurfaceCapabilitiesKHR( surface );
251-
swapChainImageFormat = chooseSwapSurfaceFormat(physicalDevice.getSurfaceFormatsKHR( surface ));
252-
swapChainExtent = chooseSwapExtent(surfaceCapabilities);
253-
auto minImageCount = std::max( 3u, surfaceCapabilities.minImageCount );
254-
minImageCount = ( surfaceCapabilities.maxImageCount > 0 && minImageCount > surfaceCapabilities.maxImageCount ) ? surfaceCapabilities.maxImageCount : minImageCount;
255-
vk::SwapchainCreateInfoKHR swapChainCreateInfo{
256-
.surface = surface, .minImageCount = minImageCount,
257-
.imageFormat = swapChainImageFormat, .imageColorSpace = vk::ColorSpaceKHR::eSrgbNonlinear,
258-
.imageExtent = swapChainExtent, .imageArrayLayers =1,
259-
.imageUsage = vk::ImageUsageFlagBits::eColorAttachment, .imageSharingMode = vk::SharingMode::eExclusive,
260-
.preTransform = surfaceCapabilities.currentTransform, .compositeAlpha = vk::CompositeAlphaFlagBitsKHR::eOpaque,
261-
.presentMode = chooseSwapPresentMode(physicalDevice.getSurfacePresentModesKHR( surface )),
262-
.clipped = true };
251+
swapChainExtent = chooseSwapExtent( surfaceCapabilities );
252+
swapChainSurfaceFormat = chooseSwapSurfaceFormat( physicalDevice.getSurfaceFormatsKHR( surface ) );
253+
vk::SwapchainCreateInfoKHR swapChainCreateInfo{ .surface = surface,
254+
.minImageCount = chooseSwapMinImageCount( surfaceCapabilities ),
255+
.imageFormat = swapChainSurfaceFormat.format,
256+
.imageColorSpace = swapChainSurfaceFormat.colorSpace,
257+
.imageExtent = swapChainExtent,
258+
.imageArrayLayers = 1,
259+
.imageUsage = vk::ImageUsageFlagBits::eColorAttachment,
260+
.imageSharingMode = vk::SharingMode::eExclusive,
261+
.preTransform = surfaceCapabilities.currentTransform,
262+
.compositeAlpha = vk::CompositeAlphaFlagBitsKHR::eOpaque,
263+
.presentMode = chooseSwapPresentMode( physicalDevice.getSurfacePresentModesKHR( surface ) ),
264+
.clipped = true };
263265

264266
swapChain = vk::raii::SwapchainKHR( device, swapChainCreateInfo );
265267
swapChainImages = swapChain.getImages();
@@ -268,7 +270,7 @@ class HelloTriangleApplication {
268270
void createImageViews() {
269271
swapChainImageViews.clear();
270272

271-
vk::ImageViewCreateInfo imageViewCreateInfo{ .viewType = vk::ImageViewType::e2D, .format = swapChainImageFormat,
273+
vk::ImageViewCreateInfo imageViewCreateInfo{ .viewType = vk::ImageViewType::e2D, .format = swapChainSurfaceFormat.format,
272274
.subresourceRange = { vk::ImageAspectFlagBits::eColor, 0, 1, 0, 1 } };
273275
for ( auto image : swapChainImages )
274276
{
@@ -281,22 +283,30 @@ class HelloTriangleApplication {
281283

282284
}
283285

284-
static vk::Format chooseSwapSurfaceFormat(const std::vector<vk::SurfaceFormatKHR>& availableFormats) {
285-
const auto formatIt = std::ranges::find_if(availableFormats,
286-
[](const auto& format) {
287-
return format.format == vk::Format::eB8G8R8A8Srgb &&
288-
format.colorSpace == vk::ColorSpaceKHR::eSrgbNonlinear;
289-
});
290-
return formatIt != availableFormats.end() ? formatIt->format : availableFormats[0].format;
286+
static uint32_t chooseSwapMinImageCount(vk::SurfaceCapabilitiesKHR const & surfaceCapabilities) {
287+
auto minImageCount = std::max( 3u, surfaceCapabilities.minImageCount );
288+
if ((0 < surfaceCapabilities.maxImageCount) && (surfaceCapabilities.maxImageCount < minImageCount)) {
289+
minImageCount = surfaceCapabilities.maxImageCount;
290+
}
291+
return minImageCount;
292+
}
293+
294+
static vk::SurfaceFormatKHR chooseSwapSurfaceFormat(const std::vector<vk::SurfaceFormatKHR>& availableFormats) {
295+
assert(!availableFormats.empty());
296+
const auto formatIt = std::ranges::find_if(
297+
availableFormats,
298+
[]( const auto & format ) { return format.format == vk::Format::eB8G8R8A8Srgb && format.colorSpace == vk::ColorSpaceKHR::eSrgbNonlinear; } );
299+
return formatIt != availableFormats.end() ? *formatIt : availableFormats[0];
291300
}
292301

293302
static vk::PresentModeKHR chooseSwapPresentMode(const std::vector<vk::PresentModeKHR>& availablePresentModes) {
303+
assert(std::ranges::any_of(availablePresentModes, [](auto presentMode){ return presentMode == vk::PresentModeKHR::eFifo; }));
294304
return std::ranges::any_of(availablePresentModes,
295305
[](const vk::PresentModeKHR value) { return vk::PresentModeKHR::eMailbox == value; } ) ? vk::PresentModeKHR::eMailbox : vk::PresentModeKHR::eFifo;
296306
}
297307

298308
vk::Extent2D chooseSwapExtent(const vk::SurfaceCapabilitiesKHR& capabilities) {
299-
if (capabilities.currentExtent.width != std::numeric_limits<uint32_t>::max()) {
309+
if (capabilities.currentExtent.width != 0xFFFFFFFF) {
300310
return capabilities.currentExtent;
301311
}
302312
int width, height;

0 commit comments

Comments
 (0)