QmlDesigner: Optimize ModelNode a little bit

The functions are doing not much but are called often. So they are a
good case for inlining.

Change-Id: I755c63287d7d518b55d4f131077b98dd0345930c
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>

src/plugins/qmldesigner/designercore/include/modelnode.h

@@ -81,7 +81,7 @@ public:
     ModelNode &operator=(const ModelNode &) = default;
     ModelNode(ModelNode &&) = default;
     ModelNode &operator=(ModelNode &&) noexcept = default;
-    ~ModelNode();
+    ~ModelNode() = default;
 
     TypeName type() const;
     QString simplifiedTypeName() const;
@@ -268,7 +268,7 @@ public:
     }
 
 private: // functions
-    Internal::InternalNodePointer internalNode() const;
+    Internal::InternalNodePointer internalNode() const { return m_internalNode; }
 
     bool hasLocked() const;
 

src/plugins/qmldesigner/designercore/model/modelnode.cpp

@@ -66,10 +66,6 @@ ModelNode::ModelNode(const ModelNode &modelNode, AbstractView *view)
     , m_view(view)
 {}
 
-/*! \brief does nothing
-*/
-ModelNode::~ModelNode() = default;
-
 /*! \brief returns the name of node which is a short cut to a property like objectName
 \return name of the node
 */
@@ -657,15 +653,6 @@ void ModelNode::destroy()
 
 //\}
 
-/*! \name Property Manipulation
- *  This functions interact with properties.
- */
-
-Internal::InternalNodePointer ModelNode::internalNode() const
-{
-    return m_internalNode;
-}
-
 /*!
 \brief returns the model of the node
 \return returns the model of the node