Check main thread assumption in evictor callback implementation

1 files changed, 7 insertions, 0 deletions

diff --git a/src/library/library.cpp b/src/library/library.cpp
index 3e915c2421..9dd115c34e 100644
--- a/src/library/library.cpp
+++ b/src/library/library.cpp
@@ -423,5 +423,12 @@ void Library::slotSetTrackTableRowHeight(int rowHeight) {
 }
 
 void Library::afterEvictedTrackFromCache(GlobalTrackCacheLocker* pCacheLocker, Track* pTrack) {
+    // The evictor callback should always be called from the main
+    // thread, because the TrackDAO with the database connection
+    // is still bound to the main thread! We have to rethink this
+    // assertion after the multi-threaded database access layer is
+    // ready!
+    // TODO: Finish multi-threaded database access layer
+    DEBUG_ASSERT(QApplication::instance()->thread() == QThread::currentThread());
     m_pTrackCollection->saveTrack(pCacheLocker, pTrack);
 }