layers: Fix 7 signed/unsigned comparison warnings on VS2015
authorMichael Mc Donnell <michael@mcdonnell.dk>
Sun, 3 Apr 2016 21:47:51 +0000 (14:47 -0700)
committerDustin Graves <dustin@lunarg.com>
Mon, 4 Apr 2016 21:06:31 +0000 (15:06 -0600)
layers/core_validation.cpp
layers/swapchain.cpp
layers/threading.cpp

index 1f99480..7cc7979 100644 (file)
@@ -1488,7 +1488,7 @@ static void collect_interface_by_location(layer_data *my_data, shader_module con
     /* find the end of the entrypoint's name string. additional zero bytes follow the actual null
        terminator, to fill out the rest of the word - so we only need to look at the last byte in
        the word to determine which word contains the terminator. */
-    auto word = 3;
+    uint32_t word = 3;
     while (entrypoint.word(word) & 0xff000000u) {
         ++word;
     }
@@ -1973,13 +1973,13 @@ static void mark_accessible_ids(shader_module const *src, spirv_inst_iter entryp
                     worklist.insert(insn.word(1)); /* image -- different operand order to above */
                     break;
                 case spv::OpFunctionCall:
-                    for (auto i = 3; i < insn.len(); i++) {
+                    for (uint32_t i = 3; i < insn.len(); i++) {
                         worklist.insert(insn.word(i)); /* fn itself, and all args */
                     }
                     break;
 
                 case spv::OpExtInst:
-                    for (auto i = 5; i < insn.len(); i++) {
+                    for (uint32_t i = 5; i < insn.len(); i++) {
                         worklist.insert(insn.word(i)); /* operands to ext inst */
                     }
                     break;
index 1feabfd..bbeb267 100644 (file)
@@ -1284,7 +1284,7 @@ static VkBool32 validateCreateSwapchainKHR(VkDevice device, const VkSwapchainCre
     // Validate pCreateInfo values with result of
     // vkGetPhysicalDeviceQueueFamilyProperties
     if (pPhysicalDevice && pPhysicalDevice->gotQueueFamilyPropertyCount) {
-        for (auto i = 0; i < pCreateInfo->queueFamilyIndexCount; i++) {
+        for (uint32_t i = 0; i < pCreateInfo->queueFamilyIndexCount; i++) {
             if (pCreateInfo->pQueueFamilyIndices[i] >= pPhysicalDevice->numOfQueueFamilies) {
                 skipCall |= LOG_ERROR_QUEUE_FAMILY_INDEX_TOO_LARGE(VK_DEBUG_REPORT_OBJECT_TYPE_PHYSICAL_DEVICE_EXT, pPhysicalDevice,
                                                                    "VkPhysicalDevice", pCreateInfo->pQueueFamilyIndices[i],
index a75bcdc..5138184 100644 (file)
@@ -310,7 +310,7 @@ vkAllocateCommandBuffers(VkDevice device, const VkCommandBufferAllocateInfo *pAl
 
     // Record mapping from command buffer to command pool
     if (VK_SUCCESS == result) {
-        for (int index = 0; index < pAllocateInfo->commandBufferCount; index++) {
+        for (uint32_t index = 0; index < pAllocateInfo->commandBufferCount; index++) {
             loader_platform_thread_lock_mutex(&threadingLock);
             command_pool_map[pCommandBuffers[index]] = pAllocateInfo->commandPool;
             loader_platform_thread_unlock_mutex(&threadingLock);
@@ -328,14 +328,14 @@ void VKAPI_CALL vkFreeCommandBuffers(VkDevice device, VkCommandPool commandPool,
     const bool lockCommandPool = false; // pool is already directly locked
     startReadObject(my_data, device);
     startWriteObject(my_data, commandPool);
-    for (int index = 0; index < commandBufferCount; index++) {
+    for (uint32_t index = 0; index < commandBufferCount; index++) {
         startWriteObject(my_data, pCommandBuffers[index], lockCommandPool);
     }
 
     pTable->FreeCommandBuffers(device, commandPool, commandBufferCount, pCommandBuffers);
     finishReadObject(my_data, device);
     finishWriteObject(my_data, commandPool);
-    for (int index = 0; index < commandBufferCount; index++) {
+    for (uint32_t index = 0; index < commandBufferCount; index++) {
         finishWriteObject(my_data, pCommandBuffers[index], lockCommandPool);
         loader_platform_thread_lock_mutex(&threadingLock);
         command_pool_map.erase(pCommandBuffers[index]);