=?UTF-8?Q?J=C3=B3zef=20Kucia=20?=: winevulkan: Downgrade ERR() to WARN() in wine_vkEnumerateInstanceExtensionProperties().

Alexandre Julliard julliard at winehq.org
Fri Jul 13 15:01:46 CDT 2018


Module: wine
Branch: master
Commit: 036376e6a5593f77f96b19ad713cbde70f458b18
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=036376e6a5593f77f96b19ad713cbde70f458b18

Author: Józef Kucia <jkucia at codeweavers.com>
Date:   Fri Jul 13 10:55:39 2018 +0200

winevulkan: Downgrade ERR() to WARN() in wine_vkEnumerateInstanceExtensionProperties().

Signed-off-by: Józef Kucia <jkucia at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/winevulkan/vulkan.c | 23 ++++++++++-------------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/dlls/winevulkan/vulkan.c b/dlls/winevulkan/vulkan.c
index f2ec38a..4e11c72 100644
--- a/dlls/winevulkan/vulkan.c
+++ b/dlls/winevulkan/vulkan.c
@@ -766,17 +766,16 @@ VkResult WINAPI wine_vkEnumerateDeviceExtensionProperties(VkPhysicalDevice phys_
 VkResult WINAPI wine_vkEnumerateInstanceExtensionProperties(const char *layer_name,
         uint32_t *count, VkExtensionProperties *properties)
 {
-    VkResult res;
-    uint32_t num_properties = 0, num_host_properties = 0;
-    VkExtensionProperties *host_properties = NULL;
+    uint32_t num_properties = 0, num_host_properties;
+    VkExtensionProperties *host_properties;
     unsigned int i, j;
+    VkResult res;
 
-    TRACE("%p %p %p\n", layer_name, count, properties);
+    TRACE("%p, %p, %p\n", layer_name, count, properties);
 
-    /* This shouldn't get called with layer_name set, the ICD loader prevents it. */
     if (layer_name)
     {
-        ERR("Layer enumeration not supported from ICD.\n");
+        WARN("Layer enumeration not supported from ICD.\n");
         return VK_ERROR_LAYER_NOT_PRESENT;
     }
 
@@ -784,14 +783,13 @@ VkResult WINAPI wine_vkEnumerateInstanceExtensionProperties(const char *layer_na
     if (res != VK_SUCCESS)
         return res;
 
-    host_properties = heap_calloc(num_host_properties, sizeof(*host_properties));
-    if (!host_properties)
+    if (!(host_properties = heap_calloc(num_host_properties, sizeof(*host_properties))))
         return VK_ERROR_OUT_OF_HOST_MEMORY;
 
     res = vk_funcs->p_vkEnumerateInstanceExtensionProperties(NULL, &num_host_properties, host_properties);
     if (res != VK_SUCCESS)
     {
-        ERR("Failed to retrieve host properties, res=%d\n", res);
+        ERR("Failed to retrieve host properties, res=%d.\n", res);
         heap_free(host_properties);
         return res;
     }
@@ -810,7 +808,7 @@ VkResult WINAPI wine_vkEnumerateInstanceExtensionProperties(const char *layer_na
 
     if (!properties)
     {
-        TRACE("Returning %u extensions\n", num_properties);
+        TRACE("Returning %u extensions.\n", num_properties);
         *count = num_properties;
         heap_free(host_properties);
         return VK_SUCCESS;
@@ -820,9 +818,8 @@ VkResult WINAPI wine_vkEnumerateInstanceExtensionProperties(const char *layer_na
     {
         if (wine_vk_instance_extension_supported(host_properties[i].extensionName))
         {
-            TRACE("Enabling extension '%s'\n", host_properties[i].extensionName);
-            properties[j] = host_properties[i];
-            j++;
+            TRACE("Enabling extension '%s'.\n", host_properties[i].extensionName);
+            properties[j++] = host_properties[i];
         }
     }
     *count = min(*count, num_properties);




More information about the wine-cvs mailing list