diff options
author | Marc Mutz <marc.mutz@qt.io> | 2023-07-21 12:32:53 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@qt.io> | 2023-08-06 11:33:03 +0000 |
commit | a9982a3b637e482c52a1ca97f9e235eb302048c3 (patch) | |
tree | 5117da43b6011e492bb4dec83fb325dbd51695ac | |
parent | 5bb4020cbe3ce0619d6046708b677469415de850 (diff) |
Add copy(SlotObjUniquePtr), use it in QHostInfoResult & SlotObjSharedPtr
It's a free function, because a) it rhymes with move(ptr) and, of
course, SlotObjUniquePtr being a std::unique_ptr, b) we can't add
member functions to it (and no, inheriting from a type whose dtor is
neither protected nor virtual is nothing you will catch yours truly
doing).
Pick-to: 6.6 6.5
Change-Id: I2026126857a7bba204d683bad118e8a2c5cb2924
Reviewed-by: Ivan Solovev <ivan.solovev@qt.io>
-rw-r--r-- | src/corelib/kernel/qobjectdefs_impl.h | 12 | ||||
-rw-r--r-- | src/network/kernel/qhostinfo_p.h | 4 |
2 files changed, 8 insertions, 8 deletions
diff --git a/src/corelib/kernel/qobjectdefs_impl.h b/src/corelib/kernel/qobjectdefs_impl.h index 68da6e04e7..a45792c5d7 100644 --- a/src/corelib/kernel/qobjectdefs_impl.h +++ b/src/corelib/kernel/qobjectdefs_impl.h @@ -452,6 +452,12 @@ namespace QtPrivate { using SlotObjUniquePtr = std::unique_ptr<QSlotObjectBase, QSlotObjectBase::Deleter>; + inline SlotObjUniquePtr copy(const SlotObjUniquePtr &other) noexcept + { + if (other) + other->ref(); + return SlotObjUniquePtr{other.get()}; + } class SlotObjSharedPtr { SlotObjUniquePtr obj; @@ -465,11 +471,7 @@ namespace QtPrivate { // (that's why (QSlotObjectBase*) ctor doesn't exisit: don't know whether that one _should_) } Q_NODISCARD_CTOR SlotObjSharedPtr(const SlotObjSharedPtr &other) noexcept - : obj(other.obj.get()) - { - if (obj) - obj->ref(); - } + : obj{copy(other.obj)} {} SlotObjSharedPtr &operator=(const SlotObjSharedPtr &other) noexcept { auto copy = other; swap(copy); return *this; } diff --git a/src/network/kernel/qhostinfo_p.h b/src/network/kernel/qhostinfo_p.h index d883a33cdd..6b755ab0c2 100644 --- a/src/network/kernel/qhostinfo_p.h +++ b/src/network/kernel/qhostinfo_p.h @@ -61,11 +61,9 @@ protected: private: QHostInfoResult(const QHostInfoResult *other) - : receiver(other->receiver), slotObj(other->slotObj.get()), // ugly, but copy the pointer... + : receiver(other->receiver), slotObj{copy(other->slotObj)}, withContextObject(other->withContextObject) { - if (slotObj) - slotObj->ref(); // ... and ref it here // cleanup if the application terminates before results are delivered connect(QCoreApplication::instance(), &QCoreApplication::aboutToQuit, this, &QObject::deleteLater); |