diff options
| author | Kevin Ottens <kevin.ottens@kdab.com> | 2016-05-19 09:34:27 +0200 |
|---|---|---|
| committer | Sean Harmer <sean.harmer@kdab.com> | 2016-05-26 11:41:19 +0000 |
| commit | 0b9b9c2a3ceeda4ba1457f91cab46cefffe0a4bc (patch) | |
| tree | 32964f29b5fcb47f4e2d22785ae5f5b54a6f928f | |
| parent | 36de2a78d2bea11ccca0810925495a6ce49affe2 (diff) | |
No need to return const QNodeId
This is especially true because: we just assume QNodeId at plenty of
places internally, thus reducing the reusability of that method
internally *and* this type is immutable anyway, provides no setter or
such.
Change-Id: I3b51387e628e327695b633860f485cf2992c94ad
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
| -rw-r--r-- | src/core/nodes/qnode.cpp | 2 | ||||
| -rw-r--r-- | src/core/nodes/qnode.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/core/nodes/qnode.cpp b/src/core/nodes/qnode.cpp index a9ccd29ba..9b718da4b 100644 --- a/src/core/nodes/qnode.cpp +++ b/src/core/nodes/qnode.cpp @@ -651,7 +651,7 @@ QNode::~QNode() /*! Returns the id that uniquely identifies the QNode instance. */ -const QNodeId QNode::id() const +QNodeId QNode::id() const { Q_D(const QNode); return d->m_id; diff --git a/src/core/nodes/qnode.h b/src/core/nodes/qnode.h index 5107b3aa8..697fcc903 100644 --- a/src/core/nodes/qnode.h +++ b/src/core/nodes/qnode.h @@ -73,7 +73,7 @@ public: explicit QNode(QNode *parent = nullptr); virtual ~QNode(); - const QNodeId id() const; + QNodeId id() const; QNode *parentNode() const; bool notificationsBlocked() const; |