Remove BuildStepList::isNull

It's not really needed. The Null-ness is equivalent to a failed
fromMap() which we execute on all usage paths anyways.

Change-Id: I72bb7fb55b7f26680fa68da8eef751ca96380ecd
Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>

1 files changed, 1 insertions, 14 deletions

diff --git a/src/plugins/projectexplorer/buildsteplist.cpp b/src/plugins/projectexplorer/buildsteplist.cpp
index 1827d60206..51b25fe204 100644
--- a/src/plugins/projectexplorer/buildsteplist.cpp
+++ b/src/plugins/projectexplorer/buildsteplist.cpp
@@ -51,8 +51,7 @@ BuildStepList::BuildStepList(QObject *parent, Core::Id id) :
 }
 
 BuildStepList::BuildStepList(QObject *parent, BuildStepList *source) :
-    ProjectConfiguration(parent, source),
-    m_isNull(source->m_isNull)
+    ProjectConfiguration(parent, source)
 {
     setDisplayName(source->displayName());
     Q_ASSERT(parent);
@@ -61,13 +60,6 @@ BuildStepList::BuildStepList(QObject *parent, BuildStepList *source) :
     // will fail to clone the buildsteps!
 }
 
-BuildStepList::BuildStepList(QObject *parent, const QVariantMap &data) :
-    ProjectConfiguration(parent, Core::Id())
-{
-    Q_ASSERT(parent);
-    m_isNull = !fromMap(data);
-}
-
 BuildStepList::~BuildStepList()
 {
     qDeleteAll(m_steps);
@@ -84,11 +76,6 @@ QVariantMap BuildStepList::toMap() const
     return map;
 }
 
-bool BuildStepList::isNull() const
-{
-    return m_isNull;
-}
-
 int BuildStepList::count() const
 {
     return m_steps.count();