QtSupport: Reduce ExtraCompiler life time to plugin life time again

Amends 6467797af2.

The created extra compilers were parented to their factories, who gained
a life time extension from 'end of plugin live' to static destruction
when the factories were made static. However, this that time the likewise
static QObjectCache in GeneratedCodeModelSupport::update might be
destroyed already, leading to a potential crash in m_cache.remove(dead).

This change here re-establishes the original timing by using the
plugin itself as parent for the extra compilers.

Change-Id: Id868b7b87f00440c67af551b71359c47a5c29cba
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>

1 files changed, 7 insertions, 4 deletions

diff --git a/src/plugins/qtsupport/uicgenerator.cpp b/src/plugins/qtsupport/uicgenerator.cpp
index dee12b9f9c..74b8355af9 100644
--- a/src/plugins/qtsupport/uicgenerator.cpp
+++ b/src/plugins/qtsupport/uicgenerator.cpp
@@ -72,8 +72,9 @@ FileNameToContentsHash UicGenerator::handleProcessFinished(Process *process)
 class UicGeneratorFactory final : public ExtraCompilerFactory
 {
 public:
-    UicGeneratorFactory() = default;
+    explicit UicGeneratorFactory(QObject *guard) : m_guard(guard) {}
 
+private:
     FileType sourceType() const final { return FileType::Form; }
 
     QString sourceTag() const final { return QLatin1String("ui"); }
@@ -82,13 +83,15 @@ public:
                           const FilePath &source,
                           const FilePaths &targets) final
     {
-        return new UicGenerator(project, source, targets, this);
+        return new UicGenerator(project, source, targets, m_guard);
     }
+
+    QObject *m_guard;
 };
 
-void setupUicGenerator()
+void setupUicGenerator(QObject *guard)
 {
-    static UicGeneratorFactory theUicGeneratorFactory;
+    static UicGeneratorFactory theUicGeneratorFactory(guard);
 }
 
 } // QtSupport::Internal