flatpak: warn if all threads were not cleaned up

cd0d8327 · Aleix Pol Gonzalez · 06821351 · cd0d8327
Commit cd0d8327 authored Jan 12, 2021 by Aleix Pol Gonzalez 🐧
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

libdiscover/backends/FlatpakBackend/FlatpakBackend.cpp libdiscover/backends/FlatpakBackend/FlatpakBackend.cpp +4 -2

No files found.
--- a/libdiscover/backends/FlatpakBackend/FlatpakBackend.cpp
+++ b/libdiscover/backends/FlatpakBackend/FlatpakBackend.cpp
@@ -108,10 +108,12 @@ FlatpakBackend::FlatpakBackend(QObject* parent)
 FlatpakBackend::~FlatpakBackend()
 {
    g_cancellable_cancel(m_cancellable);
-    m_threadPool.waitForDone(200);
-    m_threadPool.clear();
    for(auto inst : qAsConst(m_installations))
        g_object_unref(inst);
+    if (!m_threadPool.waitForDone(200)) {
+        qDebug() << "could not kill them all" << m_threadPool.activeThreadCount();
+    }
+    m_threadPool.clear();

    g_object_unref(m_cancellable);
 }