STYLE: Remove Q_ASSERT from qSlicerCLIModuleFactoryHelper::modulePaths()

Fixes #3411
Slicer · May 5, 2020 · d423ac9 · d423ac9
1 parent fc8c5c6
commit d423ac9
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/Base/QTCLI/qSlicerCLIModuleFactoryHelper.cxx b/Base/QTCLI/qSlicerCLIModuleFactoryHelper.cxx
@@ -32,10 +32,18 @@
 const QStringList qSlicerCLIModuleFactoryHelper::modulePaths()
 {
   qSlicerCoreApplication * app = qSlicerCoreApplication::application();
-  Q_ASSERT(app);
+  if (!app)
+    {
+    qCritical("qSlicerCLIModuleFactoryHelper::modulePaths failed: qSlicerCoreApplication is not instantiated");
+    return QStringList();
+    }
 
   // slicerHome shouldn't be empty
-  Q_ASSERT(!app->slicerHome().isEmpty());
+  if (!app->slicerHome().isEmpty())
+    {
+    qCritical("qSlicerCLIModuleFactoryHelper::modulePaths failed: app->slicerHome() is empty");
+    return QStringList();
+    }
 
   QStringList defaultCmdLineModulePaths;
   if (QFile::exists(app->slicerHome() + "/" + Slicer_CLIMODULES_LIB_DIR))