UI: use QScopedPointer

src/calcwidget.cpp

@@ -144,6 +144,4 @@ void CalcWidget::setupComboBox(QComboBox *comboBox, const QStringList &crypto, c
     comboBox->addItems(fiat);
 }
 
-CalcWidget::~CalcWidget() {
+CalcWidget::~CalcWidget() = default;
-    delete ui;
-}

src/calcwidget.h

@@ -31,7 +31,7 @@ private:
     void convert(bool reverse);
     void setupComboBox(QComboBox *comboBox, const QStringList &crypto, const QStringList &fiat);
 
-    Ui::CalcWidget *ui;
+    QScopedPointer<Ui::CalcWidget> ui;
     bool m_comboBoxInit = false;
 };
 

src/calcwindow.cpp

@@ -23,6 +23,4 @@ void CalcWindow::closeEvent(QCloseEvent *foo) {
     emit closed();
 }
 
-CalcWindow::~CalcWindow() {
+CalcWindow::~CalcWindow() = default;
-    delete ui;
-}

src/calcwindow.h

@@ -25,7 +25,7 @@ private:
     void closeEvent(QCloseEvent *bar) override;
 
 private:
-    Ui::CalcWindow *ui;
+    QScopedPointer<Ui::CalcWindow> ui;
 };
 
 #endif // CalcWindow_H

src/coinswidget.cpp

@@ -243,6 +243,4 @@ void CoinsWidget::thawCoins(const QVector<int> &indexes) {
     m_ctx->updateBalance();
 }
 
-CoinsWidget::~CoinsWidget() {
+CoinsWidget::~CoinsWidget() = default;
-    delete ui;
-}

src/coinswidget.h

@@ -50,7 +50,7 @@ private:
         Amount
     };
 
-    Ui::CoinsWidget *ui;
+    QScopedPointer<Ui::CoinsWidget> ui;
     QSharedPointer<AppContext> m_ctx;
 
     QMenu *m_contextMenu;

src/contactswidget.cpp

@@ -150,7 +150,4 @@ void ContactsWidget::deleteContact()
     m_model->deleteRow(m_proxyModel->mapToSource(index).row());
 }
 
-ContactsWidget::~ContactsWidget()
+ContactsWidget::~ContactsWidget() = default;
-{
-    delete ui;
-}

src/contactswidget.h

@@ -42,7 +42,7 @@ private slots:
     void showHeaderMenu(const QPoint &position);
 
 private:
-    Ui::ContactsWidget *ui;
+    QScopedPointer<Ui::ContactsWidget> ui;
     QSharedPointer<AppContext> m_ctx;
 
     QAction *m_showFullAddressesAction;

src/dialog/AboutDialog.cpp

@@ -38,7 +38,5 @@ AboutDialog::AboutDialog(QWidget *parent)
     this->adjustSize();
 }
 
-AboutDialog::~AboutDialog() {
+AboutDialog::~AboutDialog() = default;
-    delete ui;
-}
 

src/dialog/AboutDialog.h

@@ -21,7 +21,7 @@ public:
     ~AboutDialog() override;
 
 private:
-    Ui::AboutDialog *ui;
+    QScopedPointer<Ui::AboutDialog> ui;
     QStringListModel *m_model;
 };
 

src/dialog/AccountSwitcherDialog.cpp

@@ -103,6 +103,4 @@ Monero::SubaddressAccountRow* AccountSwitcherDialog::currentEntry() {
     return m_ctx->wallet->subaddressAccountModel()->entryFromIndex(index);
 }
 
-AccountSwitcherDialog::~AccountSwitcherDialog() {
+AccountSwitcherDialog::~AccountSwitcherDialog() = default;
-    delete ui;
-}

src/dialog/AccountSwitcherDialog.h

@@ -33,7 +33,7 @@ private:
 
     Monero::SubaddressAccountRow* currentEntry();
 
-    Ui::AccountSwitcherDialog *ui;
+    QScopedPointer<Ui::AccountSwitcherDialog> ui;
     QSharedPointer<AppContext> m_ctx;
     SubaddressAccountModel *m_model;
     SubaddressAccountProxyModel *m_proxyModel;

src/dialog/BalanceDialog.cpp

@@ -22,6 +22,4 @@ BalanceDialog::BalanceDialog(QWidget *parent, Wallet *wallet)
     this->adjustSize();
 }
 
-BalanceDialog::~BalanceDialog() {
+BalanceDialog::~BalanceDialog() = default;
-    delete ui;
-}

src/dialog/BalanceDialog.h

@@ -21,7 +21,7 @@ public:
     ~BalanceDialog() override;
 
 private:
-    Ui::BalanceDialog *ui;
+    QScopedPointer<Ui::BalanceDialog> ui;
 };
 
 #endif //FEATHER_BALANCEDIALOG_H

src/dialog/BroadcastTxDialog.cpp

@@ -56,6 +56,4 @@ void BroadcastTxDialog::onApiResponse(const DaemonRpc::DaemonResponse &resp) {
     }
 }
 
-BroadcastTxDialog::~BroadcastTxDialog() {
+BroadcastTxDialog::~BroadcastTxDialog() = default;
-    delete ui;
-}

src/dialog/BroadcastTxDialog.h

@@ -25,7 +25,7 @@ private slots:
     void onApiResponse(const DaemonRpc::DaemonResponse &resp);
 
 private:
-    Ui::BroadcastTxDialog *ui;
+    QScopedPointer<Ui::BroadcastTxDialog> ui;
     QSharedPointer<AppContext> m_ctx;
     UtilsNetworking *m_network;
     DaemonRpc *m_rpc;

src/dialog/CalcConfigDialog.cpp

@@ -97,6 +97,4 @@ void CalcConfigDialog::fillListWidgets() {
     setChecked(ui->list_fiat, checkedFiatCurrencies);
 }
 
-CalcConfigDialog::~CalcConfigDialog() {
+CalcConfigDialog::~CalcConfigDialog() = default;
-    delete ui;
-}

src/dialog/CalcConfigDialog.h

@@ -32,7 +32,7 @@ private:
     void fillListWidgets();
     QListWidget* getVisibleListWidget();
 
-    Ui::CalcConfigDialog *ui;
+    QScopedPointer<Ui::CalcConfigDialog> ui;
 };
 
 

src/dialog/ContactsDialog.cpp

@@ -25,10 +25,7 @@ ContactsDialog::ContactsDialog(QWidget *parent, const QString &address, const QS
     this->adjustSize();
 }
 
-ContactsDialog::~ContactsDialog()
+ContactsDialog::~ContactsDialog() = default;
-{
-    delete ui;
-}
 
 QString ContactsDialog::getAddress() {
     return m_address;

src/dialog/ContactsDialog.h

@@ -22,7 +22,7 @@ public:
     QString getName();
 
 private:
-    Ui::ContactsDialog *ui;
+    QScopedPointer<Ui::ContactsDialog> ui;
 
     QString m_address;
     QString m_name;

src/dialog/DebugInfoDialog.cpp

@@ -146,6 +146,4 @@ void DebugInfoDialog::copyToClipboad() {
     Utils::copyToClipboard(text);
 }
 
-DebugInfoDialog::~DebugInfoDialog() {
+DebugInfoDialog::~DebugInfoDialog() = default;
-    delete ui;
-}

src/dialog/DebugInfoDialog.h

@@ -25,7 +25,7 @@ private:
     void copyToClipboad();
     void updateInfo();
 
-    Ui::DebugInfoDialog *ui;
+    QScopedPointer<Ui::DebugInfoDialog> ui;
     QSharedPointer<AppContext> m_ctx;
 
     QTimer m_updateTimer;

src/dialog/InfoDialog.cpp

@@ -14,6 +14,4 @@ InfoDialog::InfoDialog(QWidget *parent, const QString &title, const QString &inf
     ui->info->setPlainText(infoData);
 }
 
-InfoDialog::~InfoDialog() {
+InfoDialog::~InfoDialog() = default;
-    delete ui;
-}

src/dialog/InfoDialog.h

@@ -19,7 +19,7 @@ public:
     ~InfoDialog() override;
 
 private:
-    Ui::InfoDialog *ui;
+    QScopedPointer<Ui::InfoDialog> ui;
 };
 
 

src/dialog/KeysDialog.cpp

@@ -22,7 +22,4 @@ KeysDialog::KeysDialog(QSharedPointer<AppContext> ctx, QWidget *parent)
     this->adjustSize();
 }
 
-KeysDialog::~KeysDialog()
+KeysDialog::~KeysDialog() = default;
-{
-    delete ui;
-}

src/dialog/KeysDialog.h

@@ -20,7 +20,7 @@ public:
     ~KeysDialog() override;
 
 private:
-    Ui::KeysDialog *ui;
+    QScopedPointer<Ui::KeysDialog> ui;
 };
 
 

src/dialog/LocalMoneroInfoDialog.cpp

@@ -46,6 +46,4 @@ void LocalMoneroInfoDialog::onGoToOffer() {
     Utils::externalLinkWarning(this, offerUrl);
 }
 
-LocalMoneroInfoDialog::~LocalMoneroInfoDialog() {
+LocalMoneroInfoDialog::~LocalMoneroInfoDialog() = default;
-    delete ui;
-}

src/dialog/LocalMoneroInfoDialog.h

@@ -26,7 +26,7 @@ private slots:
 private:
     void setLabelText(QLabel *label, LocalMoneroModel::Column column);
 
-    Ui::LocalMoneroInfoDialog *ui;
+    QScopedPointer<Ui::LocalMoneroInfoDialog> ui;
     LocalMoneroModel *m_model;
     int m_row;
 };

src/dialog/OutputInfoDialog.cpp

@@ -37,6 +37,4 @@ OutputInfoDialog::OutputInfoDialog(CoinsInfo *cInfo, QWidget *parent)
     this->adjustSize();
 }
 
-OutputInfoDialog::~OutputInfoDialog() {
+OutputInfoDialog::~OutputInfoDialog() = default;
-    delete ui;
-}

src/dialog/OutputInfoDialog.h

@@ -21,7 +21,7 @@ public:
     ~OutputInfoDialog() override;
 
 private:
-    Ui::OutputInfoDialog *ui;
+    QScopedPointer<Ui::OutputInfoDialog> ui;
 };
 
 

src/dialog/OutputSweepDialog.cpp

@@ -40,11 +40,6 @@ OutputSweepDialog::OutputSweepDialog(QWidget *parent, CoinsInfo* coin)
     this->adjustSize();
 }
 
-OutputSweepDialog::~OutputSweepDialog()
-{
-    delete ui;
-}
-
 QString OutputSweepDialog::address() {
     return m_address;
 }
@@ -55,4 +50,6 @@ bool OutputSweepDialog::churn() const {
 
 int OutputSweepDialog::outputs() const {
     return m_outputs;
-}
+}
+
+OutputSweepDialog::~OutputSweepDialog() = default;

src/dialog/OutputSweepDialog.h

@@ -24,7 +24,7 @@ public:
     int outputs() const;
 
 private:
-    Ui::OutputSweepDialog *ui;
+    QScopedPointer<Ui::OutputSweepDialog> ui;
 
     uint64_t m_amount;
 

src/dialog/PasswordChangeDialog.cpp

@@ -40,10 +40,7 @@ PasswordChangeDialog::PasswordChangeDialog(QWidget *parent, Wallet *wallet)
     this->adjustSize();
 }
 
-PasswordChangeDialog::~PasswordChangeDialog()
+PasswordChangeDialog::~PasswordChangeDialog() = default;
-{
-    delete ui;
-}
 
 void PasswordChangeDialog::passwordsMatch() {
     bool match = ui->lineEdit_newPassword->text() == ui->lineEdit_confirmPassword->text();

src/dialog/PasswordChangeDialog.h

@@ -20,7 +20,7 @@ public:
     ~PasswordChangeDialog() override;
 
 private:
-    Ui::PasswordChangeDialog *ui;
+    QScopedPointer<Ui::PasswordChangeDialog> ui;
     Wallet *m_wallet;
 
     void passwordsMatch();

src/dialog/PasswordDialog.cpp

@@ -22,7 +22,4 @@ PasswordDialog::PasswordDialog(const QString &walletName, bool incorrectPassword
     this->adjustSize();
 }
 
-PasswordDialog::~PasswordDialog()
+PasswordDialog::~PasswordDialog() = default;
-{
-    delete ui;
-}

src/dialog/PasswordDialog.h

@@ -21,7 +21,7 @@ public:
     QString password = "";
 
 private:
-    Ui::PasswordDialog *ui;
+    QScopedPointer<Ui::PasswordDialog> ui;
 };
 
 #endif //FEATHER_PASSWORDDIALOG_H

src/dialog/QrCodeDialog.cpp

@@ -27,10 +27,7 @@ QrCodeDialog::QrCodeDialog(QWidget *parent, const QrCode &qrCode, const QString
     this->adjustSize();
 }
 
-QrCodeDialog::~QrCodeDialog()
+QrCodeDialog::~QrCodeDialog() = default;
-{
-    delete ui;
-}
 
 void QrCodeDialog::setQrCode(const QrCode &qrCode) {
     m_pixmap = qrCode.toPixmap(1).scaled(500, 500, Qt::KeepAspectRatio);

src/dialog/QrCodeDialog.h

@@ -24,7 +24,7 @@ private:
     void copyImage();
     void saveImage();
 
-    Ui::QrCodeDialog *ui;
+    QScopedPointer<Ui::QrCodeDialog> ui;
     QPixmap m_pixmap;
 };
 

src/dialog/SeedDialog.cpp

@@ -62,7 +62,4 @@ void SeedDialog::setSeed(const QString &seed) {
                                        "</ul>").arg(words));
 }
 
-SeedDialog::~SeedDialog()
+SeedDialog::~SeedDialog() = default;
-{
-    delete ui;
-}

src/dialog/SeedDialog.h

@@ -22,7 +22,7 @@ public:
 private:
     void setSeed(const QString &seed);
 
-    Ui::SeedDialog *ui;
+    QScopedPointer<Ui::SeedDialog> ui;
     QSharedPointer<AppContext> m_ctx;
 };
 

src/dialog/SignVerifyDialog.cpp

@@ -52,7 +52,4 @@ void SignVerifyDialog::copyToClipboard() {
     Utils::copyToClipboard(sig.join("\n"));
 }
 
-SignVerifyDialog::~SignVerifyDialog()
+SignVerifyDialog::~SignVerifyDialog() = default;
-{
-    delete ui;
-}

src/dialog/SignVerifyDialog.h

@@ -20,7 +20,7 @@ public:
     ~SignVerifyDialog() override;
 
 private:
-    Ui::SignVerifyDialog *ui;
+    QScopedPointer<Ui::SignVerifyDialog> ui;
     Wallet *m_wallet;
 
 private slots:

src/dialog/SplashDialog.cpp

@@ -29,6 +29,4 @@ void SplashDialog::setIcon(const QPixmap &icon) {
     ui->icon->setPixmap(icon.scaledToWidth(32, Qt::SmoothTransformation));
 }
 
-SplashDialog::~SplashDialog() {
+SplashDialog::~SplashDialog() = default;
-    delete ui;
-}

src/dialog/SplashDialog.h

@@ -22,7 +22,7 @@ public:
     void setIcon(const QPixmap &icon);
 
 private:
-    Ui::SplashDialog *ui;
+    QScopedPointer<Ui::SplashDialog> ui;
 };
 
 #endif //FEATHER_SPLASHDIALOG_H

src/dialog/TorInfoDialog.cpp

@@ -165,6 +165,4 @@ void TorInfoDialog::onShowInitSyncConfigDialog() {
     }
 }
 
-TorInfoDialog::~TorInfoDialog() {
+TorInfoDialog::~TorInfoDialog() = default;
-    delete ui;
-}

src/dialog/TorInfoDialog.h

@@ -38,7 +38,7 @@ private:
     void initConnectionSettings();
     void initPrivacyLevel();
 
-    Ui::TorInfoDialog *ui;
+    QScopedPointer<Ui::TorInfoDialog> ui;
     QSharedPointer<AppContext> m_ctx;
 };
 

src/dialog/TransactionInfoDialog.cpp

@@ -134,6 +134,4 @@ void TransactionInfoDialog::createTxProof() {
     m_txProofDialog->show();
 }
 
-TransactionInfoDialog::~TransactionInfoDialog() {
+TransactionInfoDialog::~TransactionInfoDialog() = default;
-    delete ui;
-}

src/dialog/TransactionInfoDialog.h

@@ -31,7 +31,7 @@ private:
     void setData(TransactionInfo* tx);
     void updateData();
 
-    Ui::TransactionInfoDialog *ui;
+    QScopedPointer<Ui::TransactionInfoDialog> ui;
 
     QSharedPointer<AppContext> m_ctx;
     TransactionInfo *m_txInfo;

src/dialog/TxConfAdvDialog.cpp

@@ -184,6 +184,4 @@ void TxConfAdvDialog::closeDialog() {
     QDialog::reject();
 }
 
-TxConfAdvDialog::~TxConfAdvDialog() {
+TxConfAdvDialog::~TxConfAdvDialog() = default;
-    delete ui;
-}

src/dialog/TxConfAdvDialog.h

@@ -42,7 +42,7 @@ private:
     void signedQrCode();
     void signedSaveFile();
 
-    Ui::TxConfAdvDialog *ui;
+    QScopedPointer<Ui::TxConfAdvDialog> ui;
     QSharedPointer<AppContext> m_ctx;
     PendingTransaction *m_tx = nullptr;
     UnsignedTransaction *m_utx = nullptr;

src/dialog/TxConfDialog.cpp

@@ -86,6 +86,4 @@ void TxConfDialog::setShowAdvanced() {
     QDialog::reject();
 }
 
-TxConfDialog::~TxConfDialog() {
+TxConfDialog::~TxConfDialog() = default;
-    delete ui;
-}

src/dialog/TxConfDialog.h

@@ -26,7 +26,7 @@ public:
 private:
     void setShowAdvanced();
 
-    Ui::TxConfDialog *ui;
+    QScopedPointer<Ui::TxConfDialog> ui;
     QSharedPointer<AppContext> m_ctx;
     PendingTransaction *m_tx;
     QString m_address;

src/dialog/TxImportDialog.cpp

@@ -100,6 +100,4 @@ void TxImportDialog::onImport() {
     m_ctx->refreshModels();
 }
 
-TxImportDialog::~TxImportDialog() {
+TxImportDialog::~TxImportDialog() = default;
-    delete ui;
-}

src/dialog/TxImportDialog.h

@@ -26,7 +26,7 @@ private slots:
     void onApiResponse(const DaemonRpc::DaemonResponse &resp);
 
 private:
-    Ui::TxImportDialog *ui;
+    QScopedPointer<Ui::TxImportDialog> ui;
     QSharedPointer<AppContext> m_ctx;
 
     UtilsNetworking *m_network;

src/dialog/TxProofDialog.cpp

@@ -226,7 +226,5 @@ TxProof TxProofDialog::getProof() {
     return proof;
 }
 
-TxProofDialog::~TxProofDialog() {
+TxProofDialog::~TxProofDialog() = default;
-    delete ui;
-}
 

src/dialog/TxProofDialog.h

@@ -49,7 +49,7 @@ private:
     Mode m_mode;
     TransactionInfo::Direction m_direction;
 
-    Ui::TxProofDialog *ui;
+    QScopedPointer<Ui::TxProofDialog> ui;
     QSharedPointer<AppContext> m_ctx;
 };
 

src/dialog/UpdateDialog.cpp

@@ -223,6 +223,4 @@ void UpdateDialog::setStatus(const QString &msg, bool success) {
         ui->label_body->setStyleSheet("");
 }
 
-UpdateDialog::~UpdateDialog() {
+UpdateDialog::~UpdateDialog() = default;
-    delete ui;
-}

src/dialog/UpdateDialog.h

@@ -34,7 +34,7 @@ signals:
 private:
     void setStatus(const QString &msg, bool success = false);
 
-    Ui::UpdateDialog *ui;
+    QScopedPointer<Ui::UpdateDialog> ui;
 
     QString m_version;
     QString m_downloadUrl;

src/dialog/VerifyProofDialog.cpp

@@ -53,10 +53,7 @@ VerifyProofDialog::VerifyProofDialog(Wallet *wallet, QWidget *parent)
     ui->input_formattedProof->setFont(ModelUtils::getMonospaceFont());
 }
 
-VerifyProofDialog::~VerifyProofDialog()
+VerifyProofDialog::~VerifyProofDialog() = default;
-{
-    delete ui;
-}
 
 void VerifyProofDialog::checkProof() {
     switch (ui->tabWidget->currentIndex()) {

src/dialog/VerifyProofDialog.h

@@ -31,7 +31,7 @@ private:
     void checkFormattedProof();
     void proofStatus(bool success, const QString &message);
 
-    Ui::VerifyProofDialog *ui;
+    QScopedPointer<Ui::VerifyProofDialog> ui;
     Wallet *m_wallet;
 
     QPixmap m_success;

src/dialog/ViewOnlyDialog.cpp

@@ -54,7 +54,4 @@ void ViewOnlyDialog::copyToClipboad() {
     Utils::copyToClipboard(text);
 }
 
-ViewOnlyDialog::~ViewOnlyDialog()
+ViewOnlyDialog::~ViewOnlyDialog() = default;
-{
-    delete ui;
-}

src/dialog/ViewOnlyDialog.h

@@ -23,7 +23,7 @@ private slots:
     void onWriteViewOnlyWallet();
 
 private:
-    Ui::ViewOnlyDialog *ui;
+    QScopedPointer<Ui::ViewOnlyDialog> ui;
     QSharedPointer<AppContext> m_ctx;
     void copyToClipboad();
 };

src/dialog/WalletCacheDebugDialog.cpp

@@ -91,7 +91,4 @@ void WalletCacheDebugDialog::setOutput(const QString &output) {
     ui->output->setPlainText(output);
 }
 
-WalletCacheDebugDialog::~WalletCacheDebugDialog() {
+WalletCacheDebugDialog::~WalletCacheDebugDialog() = default;
-    delete ui;
-}
-

src/dialog/WalletCacheDebugDialog.h

@@ -21,7 +21,7 @@ public:
 
 private:
     void setOutput(const QString &output);
-    Ui::WalletCacheDebugDialog *ui;
+    QScopedPointer<Ui::WalletCacheDebugDialog> ui;
     QSharedPointer<AppContext> m_ctx;
 };
 

src/dialog/WalletInfoDialog.cpp

@@ -27,9 +27,7 @@ WalletInfoDialog::WalletInfoDialog(QSharedPointer<AppContext> ctx, QWidget *pare
     this->adjustSize();
 }
 
-WalletInfoDialog::~WalletInfoDialog() {
+WalletInfoDialog::~WalletInfoDialog() = default;
-    delete ui;
-}
 
 void WalletInfoDialog::openWalletDir() {
     QFileInfo file(m_ctx->wallet->keysPath());

src/dialog/WalletInfoDialog.h

@@ -23,7 +23,7 @@ public:
 private:
     void openWalletDir();
 
-    Ui::WalletInfoDialog *ui;
+    QScopedPointer<Ui::WalletInfoDialog> ui;
     QSharedPointer<AppContext> m_ctx;
 };
 

src/historywidget.cpp

@@ -181,6 +181,4 @@ void HistoryWidget::showSyncNoticeMsg() {
                              "To update the history page during synchronization press Ctrl+R.");
 }
 
-HistoryWidget::~HistoryWidget() {
+HistoryWidget::~HistoryWidget() = default;
-    delete ui;
-}

src/historywidget.h

@@ -56,7 +56,7 @@ private:
     void showContextMenu(const QPoint &point);
     void showSyncNoticeMsg();
 
-    Ui::HistoryWidget *ui;
+    QScopedPointer<Ui::HistoryWidget> ui;
     QSharedPointer<AppContext> m_ctx;
     QMenu *m_contextMenu;
     QMenu *m_copyMenu;

src/mainwindow.cpp

@@ -1424,6 +1424,4 @@ void MainWindow::toggleSearchbar(bool visible) {
         m_receiveWidget->focusSearchbar();
 }
 
-MainWindow::~MainWindow() {
+MainWindow::~MainWindow() = default;
-    delete ui;
-}

src/mainwindow.h

@@ -206,7 +206,7 @@ private:
     void updateRecentlyOpened(const QString &filename);
     void updateWidgetIcons();
 
-    Ui::MainWindow *ui;
+    QScopedPointer<Ui::MainWindow> ui;
     WindowManager *m_windowManager;
     QSharedPointer<AppContext> m_ctx;
 

src/qrcode_scanner/QrCodeScanDialog.cpp

@@ -119,5 +119,4 @@ QrCodeScanDialog::~QrCodeScanDialog()
         m_thread->terminate();
         m_thread->wait();
     }
-    delete ui;
 }

src/qrcode_scanner/QrCodeScanDialog.h

@@ -37,7 +37,7 @@ private:
     void displayCameraError();
     void takeImage();
 
-    Ui::QrCodeScanDialog *ui;
+    QScopedPointer<Ui::QrCodeScanDialog> ui;
 
     QScopedPointer<QCamera> m_camera;
     QScopedPointer<QCameraImageCapture> m_imageCapture;

src/receivewidget.cpp

@@ -243,6 +243,4 @@ Monero::SubaddressRow* ReceiveWidget::currentEntry() {
     }
 }
 
-ReceiveWidget::~ReceiveWidget() {
+ReceiveWidget::~ReceiveWidget() = default;
-    delete ui;
-}

src/receivewidget.h

@@ -50,7 +50,7 @@ private slots:
     void generateSubaddress();
 
 private:
-    Ui::ReceiveWidget *ui;
+    QScopedPointer<Ui::ReceiveWidget> ui;
     QSharedPointer<AppContext> m_ctx;
     QMenu *m_headerMenu;
     QAction *m_showFullAddressesAction;

src/sendwidget.cpp

@@ -302,6 +302,4 @@ void SendWidget::skinChanged() {
     }
 }
 
-SendWidget::~SendWidget() {
+SendWidget::~SendWidget() = default;
-    delete ui;
-}

src/sendwidget.h

@@ -48,7 +48,7 @@ private:
     void setupComboBox();
     double amountDouble();
 
-    Ui::SendWidget *ui;
+    QScopedPointer<Ui::SendWidget> ui;
     QSharedPointer<AppContext> m_ctx;
     quint64 amount();
     double conversionAmount();

src/settings.cpp

@@ -194,6 +194,4 @@ void Settings::setupLocalMoneroFrontendCombobox() {
     ui->combo_localMoneroFrontend->setCurrentIndex(ui->combo_localMoneroFrontend->findData(config()->get(Config::localMoneroFrontend).toString()));
 }
 
-Settings::~Settings() {
+Settings::~Settings() = default;
-    delete ui;
-}

src/settings.h

@@ -50,7 +50,7 @@ private:
     void setupSkinCombobox();
     void setupLocalMoneroFrontendCombobox();
 
-    Ui::Settings *ui;
+    QScopedPointer<Ui::Settings> ui;
     QSharedPointer<AppContext> m_ctx;
 
     QStringList m_skins{"Native", "QDarkStyle", "Breeze/Dark", "Breeze/Light"};

src/widgets/LocalMoneroWidget.cpp

@@ -143,10 +143,6 @@ void LocalMoneroWidget::searchOffers(int page) {
         m_api->sellMoneroOnline(currencyCode, countryCode, paymentMethod, amount, page);
 }
 
-LocalMoneroWidget::~LocalMoneroWidget() {
-    delete ui;
-}
-
 void LocalMoneroWidget::showContextMenu(const QPoint &point) {
     QModelIndex index = ui->treeView->indexAt(point);
     if (!index.isValid()) {
@@ -207,4 +203,6 @@ void LocalMoneroWidget::updatePaymentMethods() {
             ui->combo_paymentMethod->addItem(name, payment_method);
         }
     }
-}
+}
+
+LocalMoneroWidget::~LocalMoneroWidget() = default;

src/widgets/LocalMoneroWidget.h

@@ -41,7 +41,7 @@ private:
     void viewOfferDetails();
     void updatePaymentMethods();
 
-    Ui::LocalMoneroWidget *ui;
+    QScopedPointer<Ui::LocalMoneroWidget> ui;
     QSharedPointer<AppContext> m_ctx;
 
     int m_currentPage = 0;

src/widgets/RestoreHeightWidget.cpp

@@ -56,6 +56,4 @@ void RestoreHeightWidget::onRestoreHeightChanged() {
     ui->line_creationDate->setText(date.toString("yyyy-MM-dd"));
 }
 
-RestoreHeightWidget::~RestoreHeightWidget() {
+RestoreHeightWidget::~RestoreHeightWidget() = default;
-    delete ui;
-}

src/widgets/RestoreHeightWidget.h

@@ -26,7 +26,7 @@ private slots:
     void onRestoreHeightChanged();
 
 private:
-    Ui::RestoreHeightWidget *ui;
+    QScopedPointer<Ui::RestoreHeightWidget> ui;
 };
 
 

src/widgets/TickerWidget.cpp

@@ -21,9 +21,7 @@ TickerWidgetBase::TickerWidgetBase(QWidget *parent, QSharedPointer<AppContext> c
     ui->tickerFiat->setText("...");
 }
 
-TickerWidgetBase::~TickerWidgetBase() {
+TickerWidgetBase::~TickerWidgetBase() = default;
-    delete ui;
-}
 
 void TickerWidgetBase::setTitle(const QString &title) {
     ui->tickerBox->setTitle(title);

src/widgets/TickerWidget.h

@@ -30,7 +30,7 @@ public slots:
     virtual void updateDisplay() = 0;
 
 private:
-    Ui::TickerWidget *ui;
+    QScopedPointer<Ui::TickerWidget> ui;
 
 protected:
     QSharedPointer<AppContext> m_ctx;

src/widgets/ccswidget.cpp

@@ -69,6 +69,4 @@ void CCSWidget::showContextMenu(const QPoint &pos) {
     m_contextMenu->exec(ui->tableView->viewport()->mapToGlobal(pos));
 }
 
-CCSWidget::~CCSWidget() {
+CCSWidget::~CCSWidget() = default;
-    delete ui;
-}

src/widgets/ccswidget.h

@@ -40,7 +40,7 @@ private:
     void setupTable();
     void showContextMenu(const QPoint &pos);
 
-    Ui::CSSWidget *ui;
+    QScopedPointer<Ui::CSSWidget> ui;
     CCSModel *m_model;
     QMenu *m_contextMenu;
 };

src/widgets/nodewidget.cpp

@@ -188,6 +188,4 @@ NodeModel* NodeWidget::model() {
     return m_wsModel;
 }
 
-NodeWidget::~NodeWidget() {
+NodeWidget::~NodeWidget() = default;
-    delete ui;
-}

src/widgets/nodewidget.h

@@ -44,8 +44,8 @@ signals:
     void nodeSourceChanged(NodeSource nodeSource);
 
 private:
+    QScopedPointer<Ui::NodeWidget> ui;
     QSharedPointer<AppContext> m_ctx;
-    Ui::NodeWidget *ui;
     NodeModel* m_customModel;
     NodeModel* m_wsModel;
 

src/widgets/redditwidget.cpp

@@ -74,6 +74,4 @@ QString RedditWidget::getLink(const QString &permaLink) {
     return QString("https://%1%2").arg(redditFrontend, permaLink);
 }
 
-RedditWidget::~RedditWidget() {
+RedditWidget::~RedditWidget() = default;
-    delete ui;
-}

src/widgets/redditwidget.h

@@ -35,7 +35,7 @@ private:
     void copyUrl();
     QString getLink(const QString &permaLink);
 
-    Ui::RedditWidget *ui;
+    QScopedPointer<Ui::RedditWidget> ui;
     RedditModel* const m_model;
     QMenu *m_contextMenu;
 };

src/widgets/xmrigwidget.cpp

@@ -300,6 +300,4 @@ QStandardItemModel *XMRigWidget::model() {
     return m_model;
 }
 
-XMRigWidget::~XMRigWidget() {
+XMRigWidget::~XMRigWidget() = default;
-    delete ui;
-}

src/widgets/xmrigwidget.h

@@ -51,7 +51,7 @@ signals:
 private:
     void showContextMenu(const QPoint &pos);
 
-    Ui::XMRigWidget *ui;
+    QScopedPointer<Ui::XMRigWidget> ui;
     QSharedPointer<AppContext> m_ctx;
     XmRig * m_XMRig;
     QStandardItemModel *m_model;