diff options
| author | Dave Houlton <daveh@lunarg.com> | 2017-12-14 15:08:47 -0700 |
|---|---|---|
| committer | Tobin Ehlis <tobine@google.com> | 2017-12-15 09:39:50 -0700 |
| commit | acfe0d1a7d6dd1bcc7df02f4bb491d42e7abbdde (patch) | |
| tree | 665a0bcdaeaa34553f5a86f30af606f04279da2f /layers | |
| parent | 68c9b86cb62ba30f0ba869f4f5eea677bfb1c181 (diff) | |
| download | usermoji-acfe0d1a7d6dd1bcc7df02f4bb491d42e7abbdde.tar.xz | |
layers: improve const-ness in buffer_validation.cpp
Change-Id: Ic7a2ef6e26b76c1b5f3a4c279e0d15228513af9e
Diffstat (limited to 'layers')
| -rw-r--r-- | layers/buffer_validation.cpp | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/layers/buffer_validation.cpp b/layers/buffer_validation.cpp index cd13f247..b20247c2 100644 --- a/layers/buffer_validation.cpp +++ b/layers/buffer_validation.cpp @@ -1433,7 +1433,7 @@ bool ValidateCopyImageTransferGranularityRequirements(layer_data *device_data, c VkExtent3D granularity = GetScaledItg(device_data, cb_node, src_img); skip |= CheckItgOffset(device_data, cb_node, ®ion->srcOffset, &granularity, i, function, "srcOffset"); VkExtent3D subresource_extent = GetImageSubresourceExtent(src_img, ®ion->srcSubresource); - VkExtent3D extent = region->extent; + const VkExtent3D extent = region->extent; skip |= CheckItgExtent(device_data, cb_node, &extent, ®ion->srcOffset, &granularity, &subresource_extent, src_img->createInfo.imageType, i, function, "extent"); @@ -1441,7 +1441,8 @@ bool ValidateCopyImageTransferGranularityRequirements(layer_data *device_data, c granularity = GetScaledItg(device_data, cb_node, dst_img); skip |= CheckItgOffset(device_data, cb_node, ®ion->dstOffset, &granularity, i, function, "dstOffset"); // Adjust dest extent, if necessary - VkExtent3D dest_effective_extent = GetAdjustedDestImageExtent(src_img->createInfo.format, dst_img->createInfo.format, extent); + const VkExtent3D dest_effective_extent = + GetAdjustedDestImageExtent(src_img->createInfo.format, dst_img->createInfo.format, extent); subresource_extent = GetImageSubresourceExtent(dst_img, ®ion->dstSubresource); skip |= CheckItgExtent(device_data, cb_node, &dest_effective_extent, ®ion->dstOffset, &granularity, &subresource_extent, dst_img->createInfo.imageType, i, function, "extent"); @@ -1454,11 +1455,11 @@ bool ValidateImageCopyData(const layer_data *device_data, const debug_report_dat bool skip = false; for (uint32_t i = 0; i < regionCount; i++) { - VkImageCopy region = ic_regions[i]; + const VkImageCopy region = ic_regions[i]; // For comp<->uncomp copies, the copy extent for the dest image must be adjusted - VkExtent3D src_copy_extent = region.extent; - VkExtent3D dst_copy_extent = + const VkExtent3D src_copy_extent = region.extent; + const VkExtent3D dst_copy_extent = GetAdjustedDestImageExtent(src_state->createInfo.format, dst_state->createInfo.format, region.extent); bool slice_override = false; @@ -1537,7 +1538,7 @@ bool ValidateImageCopyData(const layer_data *device_data, const debug_report_dat // Checks that apply only to compressed images if (FormatIsCompressed(src_state->createInfo.format)) { - VkExtent3D block_size = FormatCompressedTexelBlockExtent(src_state->createInfo.format); + const VkExtent3D block_size = FormatCompressedTexelBlockExtent(src_state->createInfo.format); // image offsets must be multiples of block dimensions if ((SafeModulo(region.srcOffset.x, block_size.width) != 0) || @@ -1551,7 +1552,7 @@ bool ValidateImageCopyData(const layer_data *device_data, const debug_report_dat validation_error_map[VALIDATION_ERROR_09c0013a]); } - VkExtent3D mip_extent = GetImageSubresourceExtent(src_state, &(region.srcSubresource)); + const VkExtent3D mip_extent = GetImageSubresourceExtent(src_state, &(region.srcSubresource)); if ((SafeModulo(src_copy_extent.width, block_size.width) != 0) && (src_copy_extent.width + region.srcOffset.x != mip_extent.width)) { skip |= @@ -1663,7 +1664,7 @@ bool ValidateImageCopyData(const layer_data *device_data, const debug_report_dat // Checks that apply only to compressed images if (FormatIsCompressed(dst_state->createInfo.format)) { - VkExtent3D block_size = FormatCompressedTexelBlockExtent(dst_state->createInfo.format); + const VkExtent3D block_size = FormatCompressedTexelBlockExtent(dst_state->createInfo.format); // image offsets must be multiples of block dimensions if ((SafeModulo(region.dstOffset.x, block_size.width) != 0) || @@ -1677,7 +1678,7 @@ bool ValidateImageCopyData(const layer_data *device_data, const debug_report_dat validation_error_map[VALIDATION_ERROR_09c00144]); } - VkExtent3D mip_extent = GetImageSubresourceExtent(dst_state, &(region.dstSubresource)); + const VkExtent3D mip_extent = GetImageSubresourceExtent(dst_state, &(region.dstSubresource)); if ((SafeModulo(dst_copy_extent.width, block_size.width) != 0) && (dst_copy_extent.width + region.dstOffset.x != mip_extent.width)) { skip |= log_msg( @@ -1727,7 +1728,7 @@ bool PreCallValidateCmdCopyImage(layer_data *device_data, GLOBAL_CB_NODE *cb_nod VkCommandBuffer command_buffer = cb_node->commandBuffer; for (uint32_t i = 0; i < region_count; i++) { - VkImageCopy region = regions[i]; + const VkImageCopy region = regions[i]; // For comp/uncomp copies, the copy extent for the dest image must be adjusted VkExtent3D src_copy_extent = region.extent; |
