From: Jon Koops <jonkoops@gmail.com> With device sharing removed, the global DSOUND_renderers list is no longer used for lookups. Devices were still added to and removed from the list, but nothing read from it. Remove the list, the critical section protecting it, and the entry field from DirectSoundDevice. Signed-off-by: Jon Koops <jonkoops@gmail.com> --- dlls/dsound/dsound.c | 11 ----------- dlls/dsound/dsound_main.c | 12 ------------ dlls/dsound/dsound_private.h | 4 ---- 3 files changed, 27 deletions(-) diff --git a/dlls/dsound/dsound.c b/dlls/dsound/dsound.c index c6b05516d32..d4cc0cdb64b 100644 --- a/dlls/dsound/dsound.c +++ b/dlls/dsound/dsound.c @@ -202,10 +202,6 @@ static void DirectSoundDevice_destroy(DirectSoundDevice *device) if (device->mta_cookie) CoDecrementMTAUsage(device->mta_cookie); - EnterCriticalSection(&DSOUND_renderers_lock); - list_remove(&device->entry); - LeaveCriticalSection(&DSOUND_renderers_lock); - /* It is allowed to release this object even when buffers are playing */ if (device->buffers) { WARN("%d secondary buffers not released\n", device->nrofbuffers); @@ -269,13 +265,10 @@ static HRESULT DirectSoundDevice_Initialize(DirectSoundDevice ** ppDevice, LPCGU if(FAILED(hr)) return hr; - EnterCriticalSection(&DSOUND_renderers_lock); - hr = DirectSoundDevice_Create(&device); if(FAILED(hr)){ WARN("DirectSoundDevice_Create failed\n"); IMMDevice_Release(mmdevice); - LeaveCriticalSection(&DSOUND_renderers_lock); return hr; } @@ -288,7 +281,6 @@ static HRESULT DirectSoundDevice_Initialize(DirectSoundDevice ** ppDevice, LPCGU if (FAILED(hr)) { free(device); - LeaveCriticalSection(&DSOUND_renderers_lock); IMMDevice_Release(mmdevice); WARN("DSOUND_ReopenDevice failed: %08lx\n", hr); return hr; @@ -325,9 +317,6 @@ static HRESULT DirectSoundDevice_Initialize(DirectSoundDevice ** ppDevice, LPCGU SetThreadPriority(device->thread, THREAD_PRIORITY_TIME_CRITICAL); *ppDevice = device; - list_add_tail(&DSOUND_renderers, &device->entry); - - LeaveCriticalSection(&DSOUND_renderers_lock); return hr; } diff --git a/dlls/dsound/dsound_main.c b/dlls/dsound/dsound_main.c index 8936b437ba2..e2b9b81beef 100644 --- a/dlls/dsound/dsound_main.c +++ b/dlls/dsound/dsound_main.c @@ -63,16 +63,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(dsound); -struct list DSOUND_renderers = LIST_INIT(DSOUND_renderers); -CRITICAL_SECTION DSOUND_renderers_lock; -static CRITICAL_SECTION_DEBUG DSOUND_renderers_lock_debug = -{ - 0, 0, &DSOUND_renderers_lock, - { &DSOUND_renderers_lock_debug.ProcessLocksList, &DSOUND_renderers_lock_debug.ProcessLocksList }, - 0, 0, { (DWORD_PTR)(__FILE__ ": DSOUND_renderers_lock") } -}; -CRITICAL_SECTION DSOUND_renderers_lock = { &DSOUND_renderers_lock_debug, -1, 0, 0, 0, 0 }; - /* Some applications expect the GUID pointers emitted from DirectSoundCaptureEnumerate to remain * valid at least until the next time DirectSoundCaptureEnumerate is called, so we store them in * these dynamically allocated arrays. */ @@ -789,8 +779,6 @@ BOOL WINAPI DllMain(HINSTANCE hInstDLL, DWORD fdwReason, LPVOID lpvReserved) GetModuleHandleExW(GET_MODULE_HANDLE_EX_FLAG_FROM_ADDRESS, (LPCWSTR)hInstDLL, &hInstDLL); break; case DLL_PROCESS_DETACH: - if (lpvReserved) break; - DeleteCriticalSection(&DSOUND_renderers_lock); break; } return TRUE; diff --git a/dlls/dsound/dsound_private.h b/dlls/dsound/dsound_private.h index a728472b7be..244d88f7538 100644 --- a/dlls/dsound/dsound_private.h +++ b/dlls/dsound/dsound_private.h @@ -105,7 +105,6 @@ struct DirectSoundDevice IAudioRenderClient *render; HANDLE sleepev, thread; - struct list entry; }; /* reference counted buffer memory for duplicated buffer memory */ @@ -250,9 +249,6 @@ HRESULT IDirectSoundCaptureImpl_Create(IUnknown *outer_unk, REFIID riid, void ** #define STATE_CAPTURING 2 #define STATE_STOPPING 3 -extern CRITICAL_SECTION DSOUND_renderers_lock; -extern struct list DSOUND_renderers; - extern GUID *DSOUND_renderer_guids; extern GUID *DSOUND_capture_guids; -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/10919