Pass parented QObjects as QPointer

Author: daschuer

src/widget/findonwebmenufactory.cpp

@@ -11,8 +11,8 @@ namespace mixxx {
 
 namespace library {
 
-void createFindOnWebSubmenus(QMenu* pFindOnWebMenu,
-        FindOnWebLast* pFindOnWebLast,
+void createFindOnWebSubmenus(QPointer<QMenu> pFindOnWebMenu,
+        QPointer<FindOnWebLast> pFindOnWebLast,
         const Track& track) {
     make_parented<FindOnWebMenuDiscogs>(pFindOnWebMenu, pFindOnWebLast, track);
     make_parented<FindOnWebMenuLastfm>(pFindOnWebMenu, pFindOnWebLast, track);

src/widget/findonwebmenufactory.h

@@ -1,5 +1,7 @@
 #pragma once
 
+#include "QPointer"
+
 class Track;
 class QMenu;
 class FindOnWebLast;
@@ -8,8 +10,8 @@ namespace mixxx {
 
 namespace library {
 
-void createFindOnWebSubmenus(QMenu* pFindOnWebMenu,
-        FindOnWebLast* pFindOnWebLast,
+void createFindOnWebSubmenus(QPointer<QMenu> pFindOnWebMenu,
+        QPointer<FindOnWebLast> pFindOnWebLast,
         const Track& track);
 
 } // namespace library

src/widget/findonwebmenuservices/findonwebmenudiscogs.cpp

@@ -6,6 +6,7 @@
 #include "moc_findonwebmenudiscogs.cpp"
 #include "track/track.h"
 #include "util/parented_ptr.h"
+#include "widget/findonweblast.h"
 
 namespace {
 const QString kServiceTitle = QStringLiteral("Discogs");
@@ -29,8 +30,8 @@ const QUrl composeDiscogsUrl(const QString& serviceDefaultUrl,
 }
 } //namespace
 
-FindOnWebMenuDiscogs::FindOnWebMenuDiscogs(QMenu* pFindOnWebMenu,
-        FindOnWebLast* pFindOnWebLast,
+FindOnWebMenuDiscogs::FindOnWebMenuDiscogs(QPointer<QMenu> pFindOnWebMenu,
+        QPointer<FindOnWebLast> pFindOnWebLast,
         const Track& track)
         : WFindOnWebMenu(pFindOnWebMenu, pFindOnWebLast) {
     const QString artist = track.getArtist();

src/widget/findonwebmenuservices/findonwebmenudiscogs.h

@@ -9,5 +9,7 @@ class FindOnWebLast;
 class FindOnWebMenuDiscogs : public WFindOnWebMenu {
     Q_OBJECT
   public:
-    FindOnWebMenuDiscogs(QMenu* pFindOnWebMenu, FindOnWebLast* pFindOnWebLast, const Track& track);
+    FindOnWebMenuDiscogs(QPointer<QMenu> pFindOnWebMenu,
+            QPointer<FindOnWebLast> pFindOnWebLast,
+            const Track& track);
 };

src/widget/findonwebmenuservices/findonwebmenulastfm.cpp

@@ -6,6 +6,7 @@
 #include "moc_findonwebmenulastfm.cpp"
 #include "track/track.h"
 #include "util/parented_ptr.h"
+#include "widget/findonweblast.h"
 
 namespace {
 const QString kServiceTitle = QStringLiteral("LastFm");
@@ -27,8 +28,8 @@ const QUrl composeLastfmUrl(const QString& serviceSearchUrl,
 
 } //namespace
 
-FindOnWebMenuLastfm::FindOnWebMenuLastfm(QMenu* pFindOnWebMenu,
-        FindOnWebLast* pFindOnWebLast,
+FindOnWebMenuLastfm::FindOnWebMenuLastfm(QPointer<QMenu> pFindOnWebMenu,
+        QPointer<FindOnWebLast> pFindOnWebLast,
         const Track& track)
         : WFindOnWebMenu(pFindOnWebMenu, pFindOnWebLast) {
     const QString artist = track.getArtist();

src/widget/findonwebmenuservices/findonwebmenulastfm.h

@@ -9,5 +9,7 @@ class FindOnWebLast;
 class FindOnWebMenuLastfm : public WFindOnWebMenu {
     Q_OBJECT
   public:
-    FindOnWebMenuLastfm(QMenu* pFindOnWebMenu, FindOnWebLast* pFindOnWebLast, const Track& track);
+    FindOnWebMenuLastfm(QPointer<QMenu> pFindOnWebMenu,
+            QPointer<FindOnWebLast> pFindOnWebLast,
+            const Track& track);
 };

src/widget/findonwebmenuservices/findonwebmenusoundcloud.cpp

@@ -6,6 +6,7 @@
 #include "moc_findonwebmenusoundcloud.cpp"
 #include "track/track.h"
 #include "util/parented_ptr.h"
+#include "widget/findonweblast.h"
 
 namespace {
 const QString kServiceTitle = QStringLiteral("Soundcloud");
@@ -27,7 +28,7 @@ const QUrl composeSoundcloudUrl(const QString& serviceSearchUrl,
 } // namespace
 
 FindOnWebMenuSoundcloud::FindOnWebMenuSoundcloud(
-        QMenu* pFindOnWebMenu, FindOnWebLast* pFindOnWebLast, const Track& track)
+        QPointer<QMenu> pFindOnWebMenu, QPointer<FindOnWebLast> pFindOnWebLast, const Track& track)
         : WFindOnWebMenu(pFindOnWebMenu, pFindOnWebLast) {
     const QString artist = track.getArtist();
     const QString trackTitle = track.getTitle();

src/widget/findonwebmenuservices/findonwebmenusoundcloud.h

@@ -8,7 +8,7 @@ class FindOnWebLast;
 class FindOnWebMenuSoundcloud : public WFindOnWebMenu {
     Q_OBJECT
   public:
-    FindOnWebMenuSoundcloud(QMenu* pFindOnWebMenu,
-            FindOnWebLast* pFindOnWebLast,
+    FindOnWebMenuSoundcloud(QPointer<QMenu> pFindOnWebMenu,
+            QPointer<FindOnWebLast> pFindOnWebLast,
             const Track& track);
 };

src/widget/wfindonwebmenu.cpp

@@ -8,9 +8,9 @@
 #include "util/desktophelper.h"
 #include "widget/findonweblast.h"
 
-WFindOnWebMenu::WFindOnWebMenu(QWidget* parent, FindOnWebLast* pFindOnWebLast)
-        : QMenu(parent),
-          m_pFindOnWebLast(pFindOnWebLast) {
+WFindOnWebMenu::WFindOnWebMenu(QPointer<QMenu> pParent, QPointer<FindOnWebLast> pFindOnWebLast)
+        : QMenu(pParent),
+          m_pFindOnWebLast(std::move(pFindOnWebLast)) {
 }
 
 bool WFindOnWebMenu::hasEntriesForTrack(const Track& track) {

src/widget/wfindonwebmenu.h

@@ -1,14 +1,15 @@
 #pragma once
 
 #include <QMenu>
+#include <QPointer>
 
 class Track;
 class FindOnWebLast;
 
 class WFindOnWebMenu : public QMenu {
     Q_OBJECT
   public:
-    explicit WFindOnWebMenu(QWidget* pParent, FindOnWebLast* pFindOnWebLast);
+    explicit WFindOnWebMenu(QPointer<QMenu> pParent, QPointer<FindOnWebLast> pFindOnWebLast);
     ~WFindOnWebMenu() override = default;
 
     void addActionToServiceMenu(
@@ -23,5 +24,5 @@ class WFindOnWebMenu : public QMenu {
     void openInBrowser(const QUrl& url);
 
   private:
-    FindOnWebLast* m_pFindOnWebLast;
+    QPointer<FindOnWebLast> m_pFindOnWebLast;
 };