Re: [PATCH 0/7] MR4522: winewayland.drv: part 10.3: Vulkan swapchain presentation

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winewayland.drv/vulkan.c:
> -
>                           wayland_surface->window.client_rect.left;
> -
>        int client_height = wayland_surface->window.client_rect.bottom -
> -
>                            wayland_surface->window.client_rect.top;
> -
> -
>        pthread_mutex_unlock(&wayland_surface->mutex);
> -
> -
>        if (client_width != wine_vk_swapchain->extent.width ||
> -
>            client_height != wine_vk_swapchain->extent.height)
> -
>        {
> -
>            vk_result_update_severity(&res, VK_ERROR_OUT_OF_DATE_KHR);
> -
>        }
> -
>    }
> -
>    else
> -
>    {
> -
>        vk_result_update_severity(&res, VK_ERROR_SURFACE_LOST_KHR);
> -
>    }

I think you could make this simpler and return VK_ERROR_OUT_OF_DATE_KHR in both cases. This is what the Nvidia driver on Windows does afaics.