Stop worker threads before exiting

src/analyzer/trackanalysisscheduler.cpp

@@ -264,9 +264,11 @@ bool TrackAnalysisScheduler::submitNextTrack(Worker* worker) {
 
 void TrackAnalysisScheduler::stop() {
     kLogger.debug() << "Stopping";
+    m_queuedTrackIds.clear();
     for (auto& worker: m_workers) {
         worker.stopThread();
     }
+    m_workers.clear();
 }
 
 TrackPointer TrackAnalysisScheduler::loadTrackById(TrackId trackId) {

src/library/analysisfeature.cpp

@@ -54,9 +54,10 @@ AnalysisFeature::AnalysisFeature(
     setTitleDefault();
 }
 
-AnalysisFeature::~AnalysisFeature() {
-    // TODO(XXX) delete these
-    //delete m_pLibraryTableModel;
+void AnalysisFeature::stop() {
+    if (m_pTrackAnalysisScheduler) {
+        m_pTrackAnalysisScheduler->stop();
+    }
 }
 
 void AnalysisFeature::setTitleDefault() {

src/library/analysisfeature.h

@@ -26,7 +26,9 @@ class AnalysisFeature : public LibraryFeature {
   public:
     AnalysisFeature(Library* parent,
                     UserSettingsPointer pConfig);
-    virtual ~AnalysisFeature();
+    ~AnalysisFeature() override = default;
+
+    void stop();
 
     QVariant title();
     QIcon getIcon();

src/library/library.cpp

@@ -211,6 +211,14 @@ Library::~Library() {
     delete m_pTrackCollection;
 }
 
+void Library::stopFeatures() {
+    if (m_pAnalysisFeature) {
+        m_pAnalysisFeature->stop();
+        m_pAnalysisFeature = nullptr;
+    }
+    m_scanner.slotCancel();
+}
+
 void Library::bindSidebarWidget(WLibrarySidebar* pSidebarWidget) {
     m_pLibraryControl->bindSidebarWidget(pSidebarWidget);
 

src/library/library.h

@@ -51,6 +51,8 @@ class Library: public QObject,
             RecordingManager* pRecordingManager);
     ~Library() override;
 
+    void stopFeatures();
+
     mixxx::DbConnectionPoolPtr dbConnectionPool() const {
         return m_pDbConnectionPool;
     }

src/mixer/playermanager.cpp

@@ -103,7 +103,10 @@ PlayerManager::~PlayerManager() {
     delete m_pCONumMicrophones;
     delete m_pCONumAuxiliaries;
 
-    m_pTrackAnalysisScheduler.reset();
+    if (m_pTrackAnalysisScheduler) {
+        m_pTrackAnalysisScheduler->stop();
+        m_pTrackAnalysisScheduler.reset();
+    }
 }
 
 void PlayerManager::bindToLibrary(Library* pLibrary) {

src/mixxx.cpp

@@ -603,6 +603,9 @@ void MixxxMainWindow::finalize() {
         qWarning() << "WMainMenuBar was not deleted by our sendPostedEvents trick.";
     }
 
+    qDebug() << t.elapsed(false).debugMillisWithUnit() << "stopping pending Library tasks";
+    m_pLibrary->stopFeatures();
+
     // SoundManager depend on Engine and Config
     qDebug() << t.elapsed(false).debugMillisWithUnit() << "deleting SoundManager";
     delete m_pSoundManager;