-
Notifications
You must be signed in to change notification settings - Fork 8
Closed
Labels
P0Priority: CriticalPriority: CriticalbugSomething isn't workingSomething isn't workingcrashIssues involving a crashIssues involving a crashgood first issueGood for newcomersGood for newcomers
Description
Calling IEngine::stop()
(Engine::stop()
in the private implementation) should stop the project (remove all running scripts from the running scripts list).
There's also a list of finished scripts (scripts to remove). Engine::stop()
only clears the list of running scripts, but doesn't clear the list of scripts to remove. This causes a crash when trying to remove finished scripts when the list of running scripts is empty.
A simple fix should be to clear the list of scripts to remove in Engine::stop()
:
void Engine::stop()
{
m_runningScripts.clear();
m_scriptsToRemove.clear();
...
}
Metadata
Metadata
Assignees
Labels
P0Priority: CriticalPriority: CriticalbugSomething isn't workingSomething isn't workingcrashIssues involving a crashIssues involving a crashgood first issueGood for newcomersGood for newcomers