aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorflow <flowlnlnln@gmail.com>2022-08-04 17:09:32 -0300
committerflow <flowlnlnln@gmail.com>2022-08-05 11:38:46 -0300
commitd835e1d14e426ea73a66a0f8e225898598581f4a (patch)
tree6686404386703b3ab38b68e247a2ddef7912c2cd
parent362ecdb583293e09b55b95d270a72eae84510935 (diff)
downloadPrismLauncher-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.h19
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)