Re: [PATCH 0/1] MR4643: winevulkan: Update to VK spec version 1.3.272.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/winevulkan/vulkan_thunks.c:
>  init_conversion_context(ctx);
>  convert_VkRenderPassBeginInfo_win32_to_host(ctx, (const VkRenderPassBeginInfo32 *)UlongToPtr(params->pRenderPassBegin), &pRenderPassBegin_host);
>  wine_cmd_buffer_from_handle((VkCommandBuffer)UlongToPtr(params->commandBuffer))->device->funcs.p_vkCmdBeginRenderPass(wine_cmd_buffer_from_handle((VkCommandBuffer)UlongToPtr(params->commandBuffer))->host_command_buffer, &pRenderPassBegin_host, params->contents);
> - convert_VkRenderPassBeginInfo_host_to_win32(&pRenderPassBegin_host, (const VkRenderPassBeginInfo32 *)UlongToPtr(params->pRenderPassBegin));

This doesn't seem right, we shouldn't need to convert output here. I didn't look at details, I hope we don't needs to special-case it, but something like the [attached hack](/uploads/ff53c91705c13a835260982c71991022/hack.diff) avoids it.