Use Q_ASSERT instead of Q_CHECK_PTR for plain checks of pointer value

Q_CHECK_PTR (other than the name suggests) is intended only to check the
success of memory allocations.
Using it for any pointers instead can be misleading due to the
"Out of memory" log printed in case of hits.

1 files changed, 3 insertions, 3 deletions

diff --git a/src/dolphinnavigatorswidgetaction.cpp b/src/dolphinnavigatorswidgetaction.cpp
index 93d6859b1..c44b7e796 100644
--- a/src/dolphinnavigatorswidgetaction.cpp
+++ b/src/dolphinnavigatorswidgetaction.cpp
@@ -326,8 +326,8 @@ DolphinNavigatorsWidgetAction::ViewGeometriesHelper::ViewGeometriesHelper(QWidge
     : m_navigatorsWidget{navigatorsWidget}
     , m_navigatorsWidgetAction{navigatorsWidgetAction}
 {
-    Q_CHECK_PTR(navigatorsWidget);
-    Q_CHECK_PTR(navigatorsWidgetAction);
+    Q_ASSERT(navigatorsWidget);
+    Q_ASSERT(navigatorsWidgetAction);
 }
 
 bool DolphinNavigatorsWidgetAction::ViewGeometriesHelper::eventFilter(QObject *watched, QEvent *event)
@@ -351,7 +351,7 @@ bool DolphinNavigatorsWidgetAction::ViewGeometriesHelper::eventFilter(QObject *w
 
 void DolphinNavigatorsWidgetAction::ViewGeometriesHelper::setViewContainers(QWidget *primaryViewContainer, QWidget *secondaryViewContainer)
 {
-    Q_CHECK_PTR(primaryViewContainer);
+    Q_ASSERT(primaryViewContainer);
     if (m_primaryViewContainer) {
         m_primaryViewContainer->removeEventFilter(this);
     }