From 35dc239a7ecd665293ed185b528fa464a26e69a2 Mon Sep 17 00:00:00 2001
From: Kim Kulling <kim.kulling@googlemail.com>
Date: Sat, 13 Dec 2025 09:38:23 +0100
Subject: [PATCH] Fix invalid test for a valid pointer

---
 src/ModelLoading.cpp |  3 +--
 src/SceneData.h      | 14 +++++---------
 2 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/src/ModelLoading.cpp b/src/ModelLoading.cpp
index 78c38ae..95f9fda 100644
--- a/src/ModelLoading.cpp
+++ b/src/ModelLoading.cpp
@@ -59,7 +59,7 @@ ModelLoadingApp::~ModelLoadingApp() {
 }
 
 bool ModelLoadingApp::hasModel() const {
-    return mModelNode.isValid();
+    return mModelNode != nullptr;
 }
 
 void ModelLoadingApp::pushIntention() {
@@ -256,4 +256,3 @@ void ModelLoadingApp::onUpdate() {
 
     AppBase::onUpdate();
 }
-
diff --git a/src/SceneData.h b/src/SceneData.h
index 46b69d6..98ca03b 100644
--- a/src/SceneData.h
+++ b/src/SceneData.h
@@ -29,17 +29,13 @@ struct aiScene;
 namespace AssimpViewer {
 
 struct SceneData {
-    std::string mAssetName;
-    const aiScene *mScene;
+    std::string mAssetName{"none"};
+    const aiScene *mScene{nullptr};
 
-    SceneData();
+    SceneData() = default;
     ~SceneData() = default;
+    SceneData(const SceneData &) = delete;
+    SceneData &operator=(const SceneData &) = delete;
 };
 
-inline SceneData::SceneData() :
-        mAssetName("none"),
-        mScene(nullptr) {
-    // empty
-}
-
 } // namespace AssimpViewer