Leahnaya/SceneSwitcher
1
0
Fork 0
mirror of https://github.com/WarmUpTill/SceneSwitcher.git synced 2026-04-24 23:08:07 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

Rely on Qt when using GetCurrentWindowTitle() and OBS is in focus

Browse Source
This commit is contained in:
WarmUpTill 2023-08-22 19:51:23 +02:00 committed by WarmUpTill
parent 6605c19202
commit 8308506d71
1 changed files with 17 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/win/advanced-scene-switcher-win.cpp
View File

@ -179,14 +179,24 @@ void GetCurrentWindowTitle(std::string &title)
DWORD pid;
DWORD thid;
thid = GetWindowThreadProcessId(window, &pid);
// GetWindowText will freeze if the control it is reading was created
// in another thread.
// In this case it does not directly read the control.
// Instead it waits for the thread that created the control to process a
// WM_GETTEXT message.
// So if that thread is frozen in a WaitFor... call you have a deadlock.
// Calling GetWindowTitle() on the OBS windows might cause a deadlock in
// the following scenario:
//
// The thread using GetWindowTitle() will send a WM_GETTEXT message to
// the main thread that created the window and wait for it to be
// processed.
// The main thread itself might be blocked from processing the message,
// however, when it itself is waiting for the thread using
// GetWindowTitle() to return.
//
// So instead rely on Qt to get the title of the active window.
if (GetCurrentProcessId() == pid) {
title = "OBS";
auto window = QApplication::activeWindow();
if (window) {
title = window->windowTitle().toStdString();
} else {
title = "OBS";
}
return;
}
GetWindowTitle(window, title);