ProjectExplorer: Use more concise code to register extra compilers

Saves some line noise and a function.

(void) new Foo(this); is not really a new pattern in Creator code,
but a bit of a new pattern for "registering things".

Change-Id: Ie1bb409ed77bc20833fc07ca14715b9ecbc9bd5d
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>

1 files changed, 3 insertions, 6 deletions

diff --git a/src/plugins/projectexplorer/extracompiler.cpp b/src/plugins/projectexplorer/extracompiler.cpp
index 05c6535ec6..59fd6b9675 100644
--- a/src/plugins/projectexplorer/extracompiler.cpp
+++ b/src/plugins/projectexplorer/extracompiler.cpp
@@ -353,15 +353,12 @@ void ExtraCompiler::setContent(const Utils::FileName &file, const QByteArray &co
 
 ExtraCompilerFactory::ExtraCompilerFactory(QObject *parent) : QObject(parent)
 {
+    factories->append(this);
 }
 
-void ExtraCompilerFactory::registerExtraCompilerFactory(ExtraCompilerFactory *factory)
+ExtraCompilerFactory::~ExtraCompilerFactory()
 {
-    QList<ExtraCompilerFactory *> *factoryList = factories();
-    factoryList->append(factory);
-    connect(factory, &QObject::destroyed, [factoryList, factory]() {
-        factoryList->removeAll(factory);
-    });
+    factories->removeAll(this);
 }
 
 QList<ExtraCompilerFactory *> ExtraCompilerFactory::extraCompilerFactories()