From: alex-z Date: Mon, 10 Jan 2022 10:11:36 +0000 (+0200) Subject: Fix review comments. X-Git-Tag: archive/raspbian/3.16.7-1_deb13u1+rpi1~1^2~12^2~17^2~201^2~1 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=39e2292acdc48b5e87968677de6a6c5723af75a5;p=nextcloud-desktop.git Fix review comments. Signed-off-by: alex-z --- diff --git a/src/gui/passwordinputdialog.cpp b/src/gui/passwordinputdialog.cpp index 36cc9ca2a..a1febe1c3 100644 --- a/src/gui/passwordinputdialog.cpp +++ b/src/gui/passwordinputdialog.cpp @@ -19,26 +19,21 @@ namespace OCC { PasswordInputDialog::PasswordInputDialog(const QString &description, const QString &error, QWidget *parent) : QDialog(parent) - , ui(new Ui::PasswordInputDialog) + , _ui(new Ui::PasswordInputDialog) { - ui->setupUi(this); + _ui->setupUi(this); - ui->passwordLineEditLabel->setText(description); - ui->passwordLineEditLabel->setVisible(!description.isEmpty()); + _ui->passwordLineEditLabel->setText(description); + _ui->passwordLineEditLabel->setVisible(!description.isEmpty()); - ui->labelErrorMessage->setText(error); - ui->labelErrorMessage->setVisible(!error.isEmpty()); + _ui->labelErrorMessage->setText(error); + _ui->labelErrorMessage->setVisible(!error.isEmpty()); setWindowFlags(windowFlags() & ~Qt::WindowContextHelpButtonHint); } -PasswordInputDialog::~PasswordInputDialog() -{ - delete ui; -} - QString PasswordInputDialog::password() const { - return ui->passwordLineEdit->text(); + return _ui->passwordLineEdit->text(); } } diff --git a/src/gui/passwordinputdialog.h b/src/gui/passwordinputdialog.h index 151e7df74..fc7b7da6b 100644 --- a/src/gui/passwordinputdialog.h +++ b/src/gui/passwordinputdialog.h @@ -28,12 +28,11 @@ class PasswordInputDialog : public QDialog public: explicit PasswordInputDialog(const QString &description, const QString &error, QWidget *parent = nullptr); - ~PasswordInputDialog() override; QString password() const; private: - Ui::PasswordInputDialog *ui; + std::unique_ptr _ui; }; }