games/vcmi
1
0
Fork 0
mirror of https://github.com/vcmi/vcmi.git synced 2025-01-12 02:28:11 +02:00
Code Issues Releases Activity

startExecutable(): Use QProcess::start() instead of QProcess::startDetached() to fix the missing prompt after quitting vcmiclient or vcmieditor

Browse Source 
vcmilauncher is now hidden while the child process is running, since it doesn't get repainted.

Fixes #2574
This commit is contained in:
Alexander Wilms 2024-07-14 18:39:21 +02:00
parent 6f5710e809
commit 875a8cc11a
2 changed files with 9 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
launcher/main.cpp
View File

@ -108,19 +108,13 @@ void startEditor(const QStringList & args)
#ifndef VCMI_MOBILE #ifndef VCMI_MOBILE
void startExecutable(QString name, const QStringList & args) void startExecutable(QString name, const QStringList & args)
{ {
// Start vcmiclient and vcmieditor with QProcess::start() instead of QProcess::startDetached()
// since startDetached() results in a missing terminal prompt after quitting vcmiclient.
// QProcess::start() causes the launcher window to freeze while the child process is running, so we hide it in
// MainWindow::on_startGameButton_clicked() and MainWindow::on_startEditorButton_clicked()
QProcess process; QProcess process;
process.start(name, args);
// Start the executable process.waitForFinished(-1);
if(process.startDetached(name, args)) qApp->quit();
{
qApp->quit();
}
else
{
QMessageBox::critical(qApp->activeWindow(),
QObject::tr("Error starting executable"),
QObject::tr("Failed to start %1\nReason: %2").arg(name, process.errorString())
);
}
} }
#endif #endif

2
launcher/mainwindow_moc.cpp
View File

@ -196,11 +196,13 @@ MainWindow::~MainWindow()
void MainWindow::on_startGameButton_clicked() void MainWindow::on_startGameButton_clicked()
{ {
hide();
startGame({}); startGame({});
} }
void MainWindow::on_startEditorButton_clicked() void MainWindow::on_startEditorButton_clicked()
{ {
hide();
startEditor({}); startEditor({});
} }