History

Edge case when removing all scripts

Atlante45 committed 11 years ago

Commit fae59ab1 parent 18c77e11 file · +2 −5Browse files

Showing the changes to interface/src/Application.cpp (1 of 1 file). Show all

Unified · Split

interface/src/Application.cpp+2−5
@@ -3414,15 +3414,12 @@ void Application::clearScriptsBeforeRunning() { · 3413 unmodified lines
34143414 }
34153415
34163416 void Application::saveScripts() {
3417− QStringList runningScripts = getRunningScripts();
3418− if (runningScripts.isEmpty()) {
3419− return;
3420− }
3421−
34223417 // Saves all currently running user-loaded scripts
34233418 Settings settings;
34243419 settings.beginWriteArray(SETTINGS_KEY);
34253420 settings.remove("");
3421+
3422+ QStringList runningScripts = getRunningScripts();
34263423 int i = 0;
34273424 for (auto it = runningScripts.begin(); it != runningScripts.end(); ++it) {
34283425 if (getScriptEngine(*it)->isUserLoaded()) {

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t