Several fixes for 00_Images.adoc (#197)

* Use aggregate initialization

* Use aggregate initialization to match demo code and the rest of the tutorial.
Modify text to explain the aggregate initialization.

* Reduced indentation

Author: JackPilley

en/06_Texture_mapping/00_Images.adoc

@@ -153,7 +153,9 @@ The parameters for an image are specified in a `VkImageCreateInfo` struct:
 
 [,c++]
 ----
-vk::ImageCreateInfo imageInfo( {}, vk::ImageType::e2D, format, {width, height, 1}, 1, 1, vk::SampleCountFlagBits::e1, tiling, usage, vk::SharingMode::eExclusive, 0);
+vk::ImageCreateInfo imageInfo{ .imageType = vk::ImageType::e2D, .format = format,
+    .extent = {width, height, 1}, .mipLevels = 1, .arrayLayers = 1, .samples = vk::SampleCountFlagBits::e1,
+    .tiling = tiling, .usage = usage, .sharingMode = vk::SharingMode::eExclusive};
 ----
 
 The image type, specified in the `imageType` field, tells Vulkan with what kind of coordinate system the texels in the image are going to be addressed.
@@ -227,12 +229,16 @@ Create the function and move the image object creation and memory allocation to
 [,c++]
 ----
 void createImage(uint32_t width, uint32_t height, vk::Format format, vk::ImageTiling tiling, vk::ImageUsageFlags usage, vk::MemoryPropertyFlags properties, vk::raii::Image& image, vk::raii::DeviceMemory& imageMemory) {
-    vk::ImageCreateInfo imageInfo( {}, vk::ImageType::e2D, format, {width, height, 1}, 1, 1, vk::SampleCountFlagBits::e1, tiling, usage, vk::SharingMode::eExclusive, 0);
+    vk::ImageCreateInfo imageInfo{ .imageType = vk::ImageType::e2D, .format = format,
+        .extent = {width, height, 1}, .mipLevels = 1, .arrayLayers = 1,
+        .samples = vk::SampleCountFlagBits::e1, .tiling = tiling,
+        .usage = usage, .sharingMode = vk::SharingMode::eExclusive };
 
     image = vk::raii::Image( device, imageInfo );
 
     vk::MemoryRequirements memRequirements = image.getMemoryRequirements();
-    vk::MemoryAllocateInfo allocInfo( memRequirements.size, findMemoryType(memRequirements.memoryTypeBits, properties) );
+    vk::MemoryAllocateInfo allocInfo{ .allocationSize = memRequirements.size,
+                                        .memoryTypeIndex = findMemoryType(memRequirements.memoryTypeBits, properties) };
     imageMemory = vk::raii::DeviceMemory( device, allocInfo );
     image.bindMemory(imageMemory, 0);
 }
@@ -286,10 +292,10 @@ The function we're going to write now involves recording and executing a command
 [,c++]
 ----
 vk::raii::CommandBuffer beginSingleTimeCommands() {
-    vk::CommandBufferAllocateInfo allocInfo(commandPool, vk::CommandBufferLevel::ePrimary, 1);
+    vk::CommandBufferAllocateInfo allocInfo{ .commandPool = commandPool, .level = vk::CommandBufferLevel::ePrimary, .commandBufferCount = 1 };
     vk::raii::CommandBuffer commandBuffer = std::move(device.allocateCommandBuffers(allocInfo).front());
 
-    vk::CommandBufferBeginInfo beginInfo( vk::CommandBufferUsageFlagBits::eOneTimeSubmit );
+    vk::CommandBufferBeginInfo beginInfo{ .flags = vk::CommandBufferUsageFlagBits::eOneTimeSubmit };
     commandBuffer.begin(beginInfo);
 
     return commandBuffer;
@@ -298,7 +304,7 @@ vk::raii::CommandBuffer beginSingleTimeCommands() {
 void endSingleTimeCommands(vk::raii::CommandBuffer& commandBuffer) {
     commandBuffer.end();
 
-    vk::SubmitInfo submitInfo( {}, {}, {*commandBuffer});
+    vk::SubmitInfo submitInfo{ .commandBufferCount = 1, .pCommandBuffers = &*commandBuffer };
     graphicsQueue.submit(submitInfo, nullptr);
     graphicsQueue.waitIdle();
 }
@@ -334,13 +340,12 @@ There is an equivalent _buffer memory barrier_ to do this for buffers.
 
 [,c++]
 ----
-vk::ImageMemoryBarrier barrier( {}, {}, oldLayout, newLayout, {}, {}, image, { vk::ImageAspectFlagBits::eColor, 0, 1, 0, 1 } );
+vk::ImageMemoryBarrier barrier{ .oldLayout = oldLayout, .newLayout = newLayout, .image = image, .subresourceRange = { vk::ImageAspectFlagBits::eColor, 0, 1, 0, 1 } };
 ----
-
-The first two fields specify layout transition.
+`oldLayout` and `newLayout` specify the the layout transition.
 It is possible to use `VK_IMAGE_LAYOUT_UNDEFINED` as `oldLayout` if you don't care about the existing contents of the image.
 
-If you are using the barrier to transfer queue family ownership, then these two fields should be the indices of the queue families.
+If you are using the barrier to transfer queue family ownership, then `oldLayout` and `newLayout` fields should be the indices of the queue families.
 They must be set to `VK_QUEUE_FAMILY_IGNORED` if you don't want to do this (not the default value!).
 
 The `image` and `subresourceRange` specify the image that is affected and the specific part of the image.
@@ -388,7 +393,8 @@ This happens through `VkBufferImageCopy` structs:
 
 [,c++]
 ----
-vk::BufferImageCopy region( 0, 0, 0, { vk::ImageAspectFlagBits::eColor, 0, 0, 1 }, {0, 0, 0}, {width, height, 1});
+vk::BufferImageCopy region{ .bufferOffset = 0, .bufferRowLength = 0, .bufferImageHeight = 0, 
+    .imageSubresource = { vk::ImageAspectFlagBits::eColor, 0, 0, 1 }, .imageOffset = {0, 0, 0}, .imageExtent = {width, height, 1} };
 ----
 
 Most of these fields are self-explanatory.