Modernize code

GnuPG-bug-id: 4876

src/view/smartcardwidget.cpp

@@ -44,7 +44,7 @@ class PlaceHolderWidget: public QWidget
     Q_OBJECT
 public:
     explicit PlaceHolderWidget(QWidget *parent = nullptr)
-        : QWidget(parent)
+        : QWidget{parent}
     {
         auto lay = new QVBoxLayout;
         lay->addStretch(-1);
@@ -88,14 +88,14 @@ private:
 
 private:
     SmartCardWidget *const q;
-    QMap<std::pair<std::string, std::string>, QPointer<QWidget> > mCardWidgets;
+    QMap<std::pair<std::string, std::string>, QPointer<QWidget>> mCardWidgets;
     PlaceHolderWidget *mPlaceHolderWidget;
     QStackedWidget *mStack;
     QTabWidget *mTabWidget;
 };
 
 SmartCardWidget::Private::Private(SmartCardWidget *qq)
-    : q(qq)
+    : q{qq}
 {
     auto vLay = new QVBoxLayout(q);
 
@@ -185,12 +185,14 @@ void SmartCardWidget::Private::cardRemoved(const std::string &serialNumber, cons
     }
 }
 
-SmartCardWidget::SmartCardWidget(QWidget *parent):
-    QWidget(parent),
-    d(new Private(this))
+SmartCardWidget::SmartCardWidget(QWidget *parent)
+    : QWidget{parent}
+    , d{std::make_unique<Private>(this)}
 {
 }
 
+Kleo::SmartCardWidget::~SmartCardWidget() = default;
+
 void SmartCardWidget::reload()
 {
     ReaderStatus::mutableInstance()->updateStatus();

src/view/smartcardwidget.h

-#pragma once
 /*  view/smartcardwidget.h
 
     This file is part of Kleopatra, the KDE keymanager
@@ -7,25 +6,29 @@
 
     SPDX-License-Identifier: GPL-2.0-or-later
 */
+#pragma once
 
 #include <QWidget>
+
 #include <memory>
 
 namespace Kleo
 {
+
 /* SmartCardWidget a generic widget to interact with smartcards */
 class SmartCardWidget: public QWidget
 {
     Q_OBJECT
 public:
     explicit SmartCardWidget(QWidget *parent = nullptr);
+    ~SmartCardWidget() override;
 
 public Q_SLOTS:
     void reload();
 
 private:
     class Private;
-    std::shared_ptr<Private> d;
+    std::unique_ptr<Private> d;
 };
 
 } // namespace Kleo