diff --git a/src/plugins/analyzerbase/analyzersettings.cpp b/src/plugins/analyzerbase/analyzersettings.cpp index 5be75f4767075dd9721d7932b3b9e0b327f11510..325c0d4f65f55c5b9edcd3b90c4ff23c24ab95d0 100644 --- a/src/plugins/analyzerbase/analyzersettings.cpp +++ b/src/plugins/analyzerbase/analyzersettings.cpp @@ -67,18 +67,15 @@ QVariantMap AnalyzerSettings::defaults() const return map; } -bool AnalyzerSettings::fromMap(const QVariantMap &map) +void AnalyzerSettings::fromMap(const QVariantMap &map) { - return fromMap(map, &m_subConfigs); + fromMap(map, &m_subConfigs); } -bool AnalyzerSettings::fromMap(const QVariantMap &map, QList<AbstractAnalyzerSubConfig *> *subConfigs) +void AnalyzerSettings::fromMap(const QVariantMap &map, QList<AbstractAnalyzerSubConfig *> *subConfigs) { - bool ret = true; - foreach (AbstractAnalyzerSubConfig *config, *subConfigs) { - ret = ret && config->fromMap(map); - } - return ret; + foreach (AbstractAnalyzerSubConfig *config, *subConfigs) + config->fromMap(map); } QVariantMap AnalyzerSettings::toMap() const @@ -180,12 +177,10 @@ QString AnalyzerProjectSettings::displayName() const return tr("Analyzer Settings"); } -bool AnalyzerProjectSettings::fromMap(const QVariantMap &map) +void AnalyzerProjectSettings::fromMap(const QVariantMap &map) { - if (!AnalyzerSettings::fromMap(map, &m_customConfigurations)) - return false; + AnalyzerSettings::fromMap(map, &m_customConfigurations); m_useGlobalSettings = map.value(QLatin1String(useGlobalC), true).toBool(); - return true; } QVariantMap AnalyzerProjectSettings::toMap() const diff --git a/src/plugins/analyzerbase/analyzersettings.h b/src/plugins/analyzerbase/analyzersettings.h index 37a84980d8d9fd392ef8471c7a1ea9e99d168746..3a3788946d37abb7a2c03653206a199ca4815977 100644 --- a/src/plugins/analyzerbase/analyzersettings.h +++ b/src/plugins/analyzerbase/analyzersettings.h @@ -74,7 +74,7 @@ public: /// convert current configuration into map for storage virtual QVariantMap toMap() const = 0; /// read configuration from @p map - virtual bool fromMap(const QVariantMap &map) = 0; + virtual void fromMap(const QVariantMap &map) = 0; /// unique ID for this configuration virtual QString id() const = 0; @@ -113,10 +113,10 @@ public: virtual QVariantMap toMap() const; protected: - virtual bool fromMap(const QVariantMap &map); + virtual void fromMap(const QVariantMap &map); QVariantMap toMap(const QList<AbstractAnalyzerSubConfig *> &subConfigs) const; - bool fromMap(const QVariantMap &map, QList<AbstractAnalyzerSubConfig *> *subConfigs); + void fromMap(const QVariantMap &map, QList<AbstractAnalyzerSubConfig *> *subConfigs); AnalyzerSettings(QObject *parent); QList<AbstractAnalyzerSubConfig *> m_subConfigs; @@ -179,7 +179,7 @@ public: QList<AbstractAnalyzerSubConfig *> customSubConfigs() const { return m_customConfigurations; } protected: - virtual bool fromMap(const QVariantMap &map); + virtual void fromMap(const QVariantMap &map); private: bool m_useGlobalSettings; diff --git a/src/plugins/projectexplorer/runconfiguration.cpp b/src/plugins/projectexplorer/runconfiguration.cpp index 4323bbb3ea0439598cc3e135ee33d0d78d13b225..1bed8f4faf3ee045fad74c54ce0209d91383a167 100644 --- a/src/plugins/projectexplorer/runconfiguration.cpp +++ b/src/plugins/projectexplorer/runconfiguration.cpp @@ -277,7 +277,7 @@ QVariantMap DebuggerProjectSettings::toMap() const return map; } -bool DebuggerProjectSettings::fromMap(const QVariantMap &map) +void DebuggerProjectSettings::fromMap(const QVariantMap &map) { m_useCppDebugger = map.value(QLatin1String(USE_CPP_DEBUGGER_KEY), true).toBool(); if (map.value(QLatin1String(USE_QML_DEBUGGER_AUTO_KEY), false).toBool()) { @@ -286,7 +286,6 @@ bool DebuggerProjectSettings::fromMap(const QVariantMap &map) bool useQml = map.value(QLatin1String(USE_QML_DEBUGGER_KEY), false).toBool(); m_useQmlDebugger = useQml ? EnableQmlDebugger : DisableQmlDebugger; } - return true; } @@ -391,12 +390,10 @@ ProjectExplorer::Abi RunConfiguration::abi() const bool RunConfiguration::fromMap(const QVariantMap &map) { - if (!m_debuggerAspect->fromMap(map)) - return false; + m_debuggerAspect->fromMap(map); foreach (IRunConfigurationAspect *aspect, m_aspects) - if (!aspect->fromMap(map)) - return false; + aspect->fromMap(map); return ProjectConfiguration::fromMap(map); } diff --git a/src/plugins/projectexplorer/runconfiguration.h b/src/plugins/projectexplorer/runconfiguration.h index ed112e9d279950af82b64f95a5427bc9da9998e2..efa40ed285592c628174431cf86587938cc27da2 100644 --- a/src/plugins/projectexplorer/runconfiguration.h +++ b/src/plugins/projectexplorer/runconfiguration.h @@ -84,7 +84,7 @@ public: virtual QString displayName() const = 0; protected: friend class RunConfiguration; - virtual bool fromMap(const QVariantMap &map) = 0; + virtual void fromMap(const QVariantMap &map) = 0; }; class PROJECTEXPLORER_EXPORT DebuggerProjectSettings @@ -104,7 +104,7 @@ public: }; QVariantMap toMap() const; - bool fromMap(const QVariantMap &map); + void fromMap(const QVariantMap &map); QString displayName() const; diff --git a/src/plugins/valgrind/valgrindsettings.cpp b/src/plugins/valgrind/valgrindsettings.cpp index ea600611151a4f2405e9e8c2b99113319e662b64..8cd66310f2bb4e63357537219bba49aa6c7cf1c8 100644 --- a/src/plugins/valgrind/valgrindsettings.cpp +++ b/src/plugins/valgrind/valgrindsettings.cpp @@ -106,7 +106,7 @@ QVariantMap ValgrindBaseSettings::defaults() const return map; } -bool ValgrindBaseSettings::fromMap(const QVariantMap &map) +void ValgrindBaseSettings::fromMap(const QVariantMap &map) { // General setIfPresent(map, QLatin1String(valgrindExeC), &m_valgrindExecutable); @@ -132,7 +132,6 @@ bool ValgrindBaseSettings::fromMap(const QVariantMap &map) &m_visualisationMinimumInclusiveCostRatio); emit changed(); - return true; } QVariantMap ValgrindBaseSettings::toMap() const @@ -311,7 +310,7 @@ QVariantMap ValgrindGlobalSettings::defaults() const return map; } -bool ValgrindGlobalSettings::fromMap(const QVariantMap &map) +void ValgrindGlobalSettings::fromMap(const QVariantMap &map) { ValgrindBaseSettings::fromMap(map); @@ -325,8 +324,6 @@ bool ValgrindGlobalSettings::fromMap(const QVariantMap &map) if (map.contains(QLatin1String(callgrindCostFormatC))) m_costFormat = static_cast<CostDelegate::CostFormat>(map.value(QLatin1String(callgrindCostFormatC)).toInt()); setIfPresent(map, QLatin1String(callgrindCycleDetectionC), &m_detectCycles); - - return true; } QVariantMap ValgrindGlobalSettings::toMap() const @@ -441,15 +438,13 @@ QVariantMap ValgrindProjectSettings::defaults() const return map; } -bool ValgrindProjectSettings::fromMap(const QVariantMap &map) +void ValgrindProjectSettings::fromMap(const QVariantMap &map) { ValgrindBaseSettings::fromMap(map); // Memcheck setIfPresent(map, addedSuppressionFilesC, &m_addedSuppressionFiles); setIfPresent(map, removedSuppressionFilesC, &m_disabledGlobalSuppressionFiles); - - return true; } QVariantMap ValgrindProjectSettings::toMap() const diff --git a/src/plugins/valgrind/valgrindsettings.h b/src/plugins/valgrind/valgrindsettings.h index fbafa135e6b5271124120cb1932bcd7ee450db59..ef5b2621b95ae9adb21a203a72c9673b6b3e8ca1 100644 --- a/src/plugins/valgrind/valgrindsettings.h +++ b/src/plugins/valgrind/valgrindsettings.h @@ -57,7 +57,7 @@ public: virtual QVariantMap toMap() const; virtual QVariantMap defaults() const; - virtual bool fromMap(const QVariantMap &map); + virtual void fromMap(const QVariantMap &map); virtual QString id() const; virtual QString displayName() const; @@ -176,7 +176,7 @@ public: QWidget *createConfigWidget(QWidget *parent); QVariantMap toMap() const; QVariantMap defaults() const; - bool fromMap(const QVariantMap &map); + void fromMap(const QVariantMap &map); /* * Global memcheck settings @@ -231,7 +231,7 @@ public: QWidget *createConfigWidget(QWidget *parent); QVariantMap toMap() const; QVariantMap defaults() const; - bool fromMap(const QVariantMap &map); + void fromMap(const QVariantMap &map); /** * Per-project memcheck settings, saves a diff to the global suppression files list