piwalker/dolphin
1
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2024-11-14 21:37:52 -07:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #11306 from AdmiralCurtiss/cubeb-coinit-fixes

Browse Source 
Correctly call CoUninitialize() on Cubeb helper class destruction.
This commit is contained in:
Admiral H. Curtiss 2022-11-27 04:42:25 +01:00 committed by GitHub
commit c8d30968f3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Source/Core/AudioCommon/CubebStream.cpp
View File

@ -46,6 +46,7 @@ CubebStream::CubebStream()
Common::ScopeGuard sync_event_guard([&sync_event] { sync_event.Set(); });
auto result = ::CoInitializeEx(nullptr, COINIT_MULTITHREADED | COINIT_DISABLE_OLE1DDE);
m_coinit_success = result == S_OK;
m_should_couninit = result == S_OK || result == S_FALSE;
});
sync_event.Wait();
}
@ -137,11 +138,12 @@ CubebStream::~CubebStream()
cubeb_stream_stop(m_stream);
cubeb_stream_destroy(m_stream);
#ifdef _WIN32
if (m_coinit_success)
if (m_should_couninit)
{
m_coinit_success = false;
m_should_couninit = false;
CoUninitialize();
}
m_coinit_success = false;
});
sync_event.Wait();
#endif

1
Source/Core/AudioCommon/CubebStream.h
View File

@ -37,6 +37,7 @@ private:
#ifdef _WIN32
Common::WorkQueueThread<std::function<void()>> m_work_queue;
bool m_coinit_success = false;
bool m_should_couninit = false;
#endif
static long DataCallback(cubeb_stream* stream, void* user_data, const void* /*input_buffer*/,

15
Source/Core/Core/HW/EXI/EXI_DeviceMic.cpp
View File

@ -221,6 +221,7 @@ CEXIMic::CEXIMic(int index)
Common::ScopeGuard sync_event_guard([&sync_event] { sync_event.Set(); });
auto result = ::CoInitializeEx(nullptr, COINIT_MULTITHREADED | COINIT_DISABLE_OLE1DDE);
m_coinit_success = result == S_OK;
m_should_couninit = result == S_OK || result == S_FALSE;
});
sync_event.Wait();
#endif
@ -231,6 +232,20 @@ CEXIMic::CEXIMic(int index)
CEXIMic::~CEXIMic()
{
StreamTerminate();
#ifdef _WIN32
if (m_should_couninit)
{
Common::Event sync_event;
m_work_queue.EmplaceItem([this, &sync_event] {
Common::ScopeGuard sync_event_guard([&sync_event] { sync_event.Set(); });
m_should_couninit = false;
CoUninitialize();
});
sync_event.Wait();
}
m_coinit_success = false;
#endif
}
bool CEXIMic::IsPresent() const

1
Source/Core/Core/HW/EXI/EXI_DeviceMic.h
View File

@ -104,6 +104,7 @@ private:
#ifdef _WIN32
Common::WorkQueueThread<std::function<void()>> m_work_queue;
bool m_coinit_success = false;
bool m_should_couninit = false;
#endif
};
} // namespace ExpansionInterface