ToolChainFactories: Remove some code that is not necessary

Change-Id: I2cd3940960bad385bc2bf70959df57847dcd101f
Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>

src/plugins/projectexplorer/customtoolchain.cpp

@@ -383,7 +383,7 @@ bool CustomToolChainFactory::canCreate()
 
 ToolChain *CustomToolChainFactory::create()
 {
-    return createToolChain(false);
+    return new CustomToolChain(ToolChain::ManualDetection);
 }
 
 // Used by the ToolChainManager to restore user-generated tool chains
@@ -402,11 +402,6 @@ ToolChain *CustomToolChainFactory::restore(const QVariantMap &data)
     return 0;
 }
 
-CustomToolChain *CustomToolChainFactory::createToolChain(bool autoDetect)
-{
-    return new CustomToolChain(autoDetect ? ToolChain::AutoDetection : ToolChain::ManualDetection);
-}
-
 // --------------------------------------------------------------------------
 // Helper for ConfigWidget
 // --------------------------------------------------------------------------

src/plugins/projectexplorer/customtoolchain.h

@@ -159,11 +159,6 @@ public:
     // Used by the ToolChainManager to restore user-generated tool chains
     bool canRestore(const QVariantMap &data) override;
     ToolChain *restore(const QVariantMap &data) override;
-
-protected:
-    virtual CustomToolChain *createToolChain(bool autoDetect);
-    QList<ToolChain *> autoDetectToolchains(const QString &compiler,
-                                            const Abi &);
 };
 
 // --------------------------------------------------------------------------

src/plugins/projectexplorer/gcctoolchain.cpp

@@ -795,10 +795,8 @@ bool GccToolChainFactory::canRestore(const QVariantMap &data)
 
 ToolChain *GccToolChainFactory::restore(const QVariantMap &data)
 {
-    GccToolChain *tc = new GccToolChain(ToolChain::ManualDetection);
+    GccToolChain *tc = createToolChain(false);
-    // Updating from 2.5:
+    if (tc->fromMap(data))
-    QVariantMap updated = data;
-    if (tc->fromMap(updated))
         return tc;
 
     delete tc;
@@ -1116,31 +1114,11 @@ QList<ToolChain *> ClangToolChainFactory::autoDetect()
     return autoDetectToolchains(QLatin1String("clang++"), ha);
 }
 
-bool ClangToolChainFactory::canCreate()
-{
-    return true;
-}
-
-ToolChain *ClangToolChainFactory::create()
-{
-    return createToolChain(false);
-}
-
 bool ClangToolChainFactory::canRestore(const QVariantMap &data)
 {
     return typeIdFromMap(data) == Constants::CLANG_TOOLCHAIN_TYPEID;
 }
 
-ToolChain *ClangToolChainFactory::restore(const QVariantMap &data)
-{
-    ClangToolChain *tc = new ClangToolChain(ToolChain::ManualDetection);
-    if (tc->fromMap(data))
-        return tc;
-
-    delete tc;
-    return 0;
-}
-
 GccToolChain *ClangToolChainFactory::createToolChain(bool autoDetect)
 {
     return new ClangToolChain(autoDetect ? ToolChain::AutoDetection : ToolChain::ManualDetection);
@@ -1217,31 +1195,11 @@ QList<ToolChain *> MingwToolChainFactory::autoDetect()
                                 Abi(ha.architecture(), Abi::WindowsOS, Abi::WindowsMSysFlavor, Abi::PEFormat, ha.wordWidth()));
 }
 
-bool MingwToolChainFactory::canCreate()
-{
-    return true;
-}
-
-ToolChain *MingwToolChainFactory::create()
-{
-    return createToolChain(false);
-}
-
 bool MingwToolChainFactory::canRestore(const QVariantMap &data)
 {
     return typeIdFromMap(data) == Constants::MINGW_TOOLCHAIN_TYPEID;
 }
 
-ToolChain *MingwToolChainFactory::restore(const QVariantMap &data)
-{
-    MingwToolChain *tc = new MingwToolChain(ToolChain::ManualDetection);
-    if (tc->fromMap(data))
-        return tc;
-
-    delete tc;
-    return 0;
-}
-
 GccToolChain *MingwToolChainFactory::createToolChain(bool autoDetect)
 {
     return new MingwToolChain(autoDetect ? ToolChain::AutoDetection : ToolChain::ManualDetection);
@@ -1313,26 +1271,11 @@ QList<ToolChain *> LinuxIccToolChainFactory::autoDetect()
     return autoDetectToolchains(QLatin1String("icpc"), Abi::hostAbi());
 }
 
-ToolChain *LinuxIccToolChainFactory::create()
-{
-    return createToolChain(false);
-}
-
 bool LinuxIccToolChainFactory::canRestore(const QVariantMap &data)
 {
     return typeIdFromMap(data) == Constants::LINUXICC_TOOLCHAIN_TYPEID;
 }
 
-ToolChain *LinuxIccToolChainFactory::restore(const QVariantMap &data)
-{
-    LinuxIccToolChain *tc = new LinuxIccToolChain(ToolChain::ManualDetection);
-    if (tc->fromMap(data))
-        return tc;
-
-    delete tc;
-    return 0;
-}
-
 GccToolChain *LinuxIccToolChainFactory::createToolChain(bool autoDetect)
 {
     return new LinuxIccToolChain(autoDetect ? ToolChain::AutoDetection : ToolChain::ManualDetection);

src/plugins/projectexplorer/gcctoolchainfactories.h

@@ -116,11 +116,7 @@ public:
 
     QList<ToolChain *> autoDetect();
 
-    bool canCreate();
-    ToolChain *create();
-
     bool canRestore(const QVariantMap &data);
-    ToolChain *restore(const QVariantMap &data);
 
 protected:
     GccToolChain *createToolChain(bool autoDetect);
@@ -139,11 +135,7 @@ public:
 
     QList<ToolChain *> autoDetect();
 
-    bool canCreate();
-    ToolChain *create();
-
     bool canRestore(const QVariantMap &data);
-    ToolChain *restore(const QVariantMap &data);
 
 protected:
     GccToolChain *createToolChain(bool autoDetect);
@@ -162,10 +154,7 @@ public:
 
     QList<ToolChain *> autoDetect();
 
-    ToolChain *create();
-
     bool canRestore(const QVariantMap &data);
-    ToolChain *restore(const QVariantMap &data);
 
 protected:
     GccToolChain *createToolChain(bool autoDetect);