Giovanni Mascellani : dxgi: Remove the worker event, which is now useless.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 060e9f7694d9d9af38421eda0071e03021bf2b3c
URL:    https://gitlab.winehq.org/wine/wine/-/commit/060e9f7694d9d9af38421eda0071e03...

Author: Giovanni Mascellani 
Date:   Fri Jun 23 16:21:33 2023 +0200

dxgi: Remove the worker event, which is now useless.

---

dlls/dxgi/swapchain.c | 19 -------------------
 1 file changed, 19 deletions(-)

diff --git a/dlls/dxgi/swapchain.c b/dlls/dxgi/swapchain.c
index ea93efb5643..81cb6dcd575 100644
--- a/dlls/dxgi/swapchain.c
+++ b/dlls/dxgi/swapchain.c
@@ -1029,7 +1029,6 @@ struct d3d12_swapchain
     CONDITION_VARIABLE worker_cv;
     bool worker_running;
     struct list worker_ops;
-    HANDLE worker_event;

/* D3D12 side of the swapchain (frontend): these objects are
      * visible to the IDXGISwapChain client, so they must never be
@@ -1159,9 +1158,6 @@ static DWORD WINAPI d3d12_swapchain_worker_proc(void *data)

d3d12_swapchain_op_destroy(swapchain, op);

-            if (!SetEvent(swapchain->worker_event))
-                ERR("Cannot set event, last error %ld.\n", GetLastError());
-
             EnterCriticalSection(&swapchain->worker_cs);
         }
         else if (!SleepConditionVariableCS(&swapchain->worker_cv, &swapchain->worker_cs, INFINITE))
@@ -1931,9 +1927,6 @@ static void d3d12_swapchain_destroy(struct d3d12_swapchain *swapchain)
             ERR("Failed to close worker thread, last error %ld.\n", GetLastError());
     }

-    if (!CloseHandle(swapchain->worker_event))
-        ERR("Failed to close worker event, last error %ld.\n", GetLastError());
-
     DeleteCriticalSection(&swapchain->worker_cs);

LIST_FOR_EACH_ENTRY_SAFE(op, op2, &swapchain->worker_ops, struct d3d12_swapchain_op, entry)
@@ -2243,8 +2236,6 @@ static HRESULT d3d12_swapchain_present(struct d3d12_swapchain *swapchain,
     WakeAllConditionVariable(&swapchain->worker_cv);
     LeaveCriticalSection(&swapchain->worker_cs);

-    WaitForSingleObject(swapchain->worker_event, INFINITE);
-
     ++swapchain->frame_number;
     if ((frame_latency_event = swapchain->frame_latency_event))
     {
@@ -2523,8 +2514,6 @@ static HRESULT d3d12_swapchain_resize_buffers(struct d3d12_swapchain *swapchain,
     WakeAllConditionVariable(&swapchain->worker_cv);
     LeaveCriticalSection(&swapchain->worker_cs);

-    WaitForSingleObject(swapchain->worker_event, INFINITE);
-
     swapchain->current_buffer_index = 0;

d3d12_swapchain_destroy_d3d12_resources(swapchain);
@@ -3194,14 +3183,6 @@ static HRESULT d3d12_swapchain_init(struct d3d12_swapchain *swapchain, IWineDXGI
     swapchain->worker_running = true;
     list_init(&swapchain->worker_ops);

-    if (!(swapchain->worker_event = CreateEventA(NULL, FALSE, FALSE, NULL)))
-    {
-        hr = HRESULT_FROM_WIN32(GetLastError());
-        WARN("Failed to create worker event, hr %#lx.\n", hr);
-        d3d12_swapchain_destroy(swapchain);
-        return hr;
-    }
-
     surface_desc.sType = VK_STRUCTURE_TYPE_WIN32_SURFACE_CREATE_INFO_KHR;
     surface_desc.pNext = NULL;
     surface_desc.flags = 0;