History

Merge pull request #4202 from Atlante45/master

Edge case when removing all scripts

samcake committed 11 years ago

Commit efbf0432 parents 1c89e27 fae59ab1 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
@@ -3413,15 +3413,12 @@ void Application::clearScriptsBeforeRunning() { · 3412 unmodified lines
34133413 }
34143414
34153415 void Application::saveScripts() {
3416− QStringList runningScripts = getRunningScripts();
3417− if (runningScripts.isEmpty()) {
3418− return;
3419− }
3420−
34213416 // Saves all currently running user-loaded scripts
34223417 Settings settings;
34233418 settings.beginWriteArray(SETTINGS_KEY);
34243419 settings.remove("");
3420+
3421+ QStringList runningScripts = getRunningScripts();
34253422 int i = 0;
34263423 for (auto it = runningScripts.begin(); it != runningScripts.end(); ++it) {
34273424 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