refactor: CoreProcess::start, early return if process is already started
This commit is contained in:
committed by
Chris Rizzitello
parent
8d577aaa93
commit
9e94a4fe0e
@ -317,6 +317,11 @@ void CoreProcess::start(std::optional<ProcessMode> processModeOption)
|
||||
{
|
||||
using enum Settings::CoreMode;
|
||||
|
||||
if (m_processState == ProcessState::Started) {
|
||||
qCritical("core process already started");
|
||||
return;
|
||||
}
|
||||
|
||||
QMutexLocker locker(&m_processMutex);
|
||||
|
||||
const auto currentMode = Settings::value(Settings::Core::ProcessMode).value<ProcessMode>();
|
||||
@ -325,11 +330,6 @@ void CoreProcess::start(std::optional<ProcessMode> processModeOption)
|
||||
qInfo().noquote(
|
||||
) << QString("starting core %1 process (%2 mode)").arg(modeString(), processModeToString(processMode));
|
||||
|
||||
if (m_processState == ProcessState::Started) {
|
||||
qCritical("core process already started");
|
||||
return;
|
||||
}
|
||||
|
||||
// allow external listeners to abort the start process (e.g. licensing issue).
|
||||
setProcessState(ProcessState::Starting);
|
||||
Q_EMIT starting();
|
||||
|
||||
Reference in New Issue
Block a user