Minor cleanups

Author: dail8859

src/NotepadNext/NotepadNextApplication.cpp

@@ -442,11 +442,13 @@ void NotepadNextApplication::saveSettings()
 
 void NotepadNextApplication::saveSession()
 {
+    // Iterate all the opened editors and add them to the recent file list
     for (const auto &editor : window->editors()) {
         if (editor->isFile()) {
             recentFilesListManager->addFile(editor->getFilePath());
         }
     }
+    saveSettings();
 
     getSessionManager()->saveSession(window);
 }
@@ -462,13 +464,14 @@ MainWindow *NotepadNextApplication::createNewWindow()
         LuaExtension::Instance().setEditor(editor);
     });
 
-    // Since these editors don't actually get "closed" go ahead and add them to the recent file list
+    // TODO: this shouldn't be dependent on a MainWindow closing, but this works for now
+    // since the assumption is MainWindow::aboutToClose() infers the application is shutting
+    // down but the editors are still active.
     connect(window, &MainWindow::aboutToClose, this, &NotepadNextApplication::saveSession);
 
-    // Timer to autosave session each minute
+    // Timer to autosave the session
     connect(&autoSaveTimer, &QTimer::timeout, this, &NotepadNextApplication::saveSession);
-    autoSaveTimer.setInterval(60 * 1000);
-    autoSaveTimer.start();
+    autoSaveTimer.start(60 * 1000);
 
     return window;
 }