diff options
author | flow <flowlnlnln@gmail.com> | 2022-08-04 17:09:32 -0300 |
---|---|---|
committer | flow <flowlnlnln@gmail.com> | 2022-08-05 11:38:46 -0300 |
commit | d835e1d14e426ea73a66a0f8e225898598581f4a (patch) | |
tree | 6686404386703b3ab38b68e247a2ddef7912c2cd | |
parent | 362ecdb583293e09b55b95d270a72eae84510935 (diff) | |
download | PrismLauncher-d835e1d14e426ea73a66a0f8e225898598581f4a.tar.gz PrismLauncher-d835e1d14e426ea73a66a0f8e225898598581f4a.tar.bz2 PrismLauncher-d835e1d14e426ea73a66a0f8e225898598581f4a.zip |
refactor: simplify smart pointers more
Signed-off-by: flow <flowlnlnln@gmail.com>
-rw-r--r-- | launcher/QObjectPtr.h | 19 |
1 files changed, 4 insertions, 15 deletions
diff --git a/launcher/QObjectPtr.h b/launcher/QObjectPtr.h index 7c453adb..b1ef1c8d 100644 --- a/launcher/QObjectPtr.h +++ b/launcher/QObjectPtr.h @@ -2,27 +2,16 @@ #include <QObject> #include <QSharedPointer> + #include <functional> #include <memory> -namespace details { -[[maybe_unused]] static void do_delete_later(QObject* obj) -{ - if (obj) - obj->deleteLater(); -} -struct DeleteQObjectLater { - void operator()(QObject* obj) const { do_delete_later(obj); } -}; - -} // namespace details - /** * A unique pointer class with unique pointer semantics intended for derivates of QObject * Calls deleteLater() instead of destroying the contained object immediately */ template <typename T> -using unique_qobject_ptr = std::unique_ptr<T, details::DeleteQObjectLater>; +using unique_qobject_ptr = QScopedPointer<T, QScopedPointerDeleteLater>; /** * A shared pointer class with shared pointer semantics intended for derivates of QObject @@ -32,8 +21,8 @@ template <typename T> class shared_qobject_ptr : public QSharedPointer<T> { public: constexpr shared_qobject_ptr() : QSharedPointer<T>() {} - constexpr shared_qobject_ptr(T* ptr) : QSharedPointer<T>(ptr, details::do_delete_later) {} - constexpr shared_qobject_ptr(std::nullptr_t null_ptr) : QSharedPointer<T>(null_ptr, details::do_delete_later) {} + constexpr shared_qobject_ptr(T* ptr) : QSharedPointer<T>(ptr, &QObject::deleteLater) {} + constexpr shared_qobject_ptr(std::nullptr_t null_ptr) : QSharedPointer<T>(null_ptr, &QObject::deleteLater) {} template <typename Derived> constexpr shared_qobject_ptr(const shared_qobject_ptr<Derived>& other) : QSharedPointer<T>(other) |