Modernize code

src/filter/filteractions/filteraction.h

@@ -182,8 +182,8 @@ Q_SIGNALS:
     void filterActionModified();
 
 private:
-    QString mName;
-    QString mLabel;
+    const QString mName;
+    const QString mLabel;
 };
 }
 

src/filter/filterconverter/filterconverttosieve.h

@@ -20,7 +20,7 @@ public:
     void convert();
 
 private:
-    QVector<MailFilter *> mListFilters;
+    const QVector<MailFilter *> mListFilters;
 };
 }
 #endif // FILTERCONVERTTOSIEVE_H

src/filter/mdnadvicedialog.cpp

@@ -85,7 +85,6 @@ static const int numMdnMessageBoxes
 
 MDNAdviceDialog::MDNAdviceDialog(const QString &text, bool canDeny, QWidget *parent)
     : QDialog(parent)
-    , m_result(MessageComposer::MDNIgnore)
 {
     setWindowTitle(i18nc("@title:window", "Message Disposition Notification Request"));
     QPushButton *user1Button = nullptr;

src/filter/mdnadvicedialog.h

@@ -76,7 +76,7 @@ private:
     void slotUser1Clicked();
     void slotUser2Clicked();
     void slotYesClicked();
-    MessageComposer::MDNAdvice m_result;
+    MessageComposer::MDNAdvice m_result = MessageComposer::MDNIgnore;
 
 protected:
 

src/kernel/mailkernel.cpp

@@ -39,7 +39,7 @@ public:
         delete kernel;
     }
 
-    Kernel *kernel;
+    Kernel *const kernel;
 };
 
 Q_GLOBAL_STATIC(KernelPrivate, sInstance)

src/search/widgethandler/rulewidgethandlermanager.cpp

@@ -33,7 +33,6 @@ using namespace MailCommon;
 MailCommon::RuleWidgetHandlerManager *MailCommon::RuleWidgetHandlerManager::self = nullptr;
 
 MailCommon::RuleWidgetHandlerManager::RuleWidgetHandlerManager()
-    : mIsBalooSearch(false)
 {
     registerHandler(new MailCommon::TagRuleWidgetHandler());
     registerHandler(new MailCommon::DateRuleWidgetHandler());

src/search/widgethandler/rulewidgethandlermanager.h

@@ -66,7 +66,7 @@ private:
     typedef QVector<const RuleWidgetHandler *>::iterator iterator;
 
     QVector<const RuleWidgetHandler *> mHandlers;
-    bool mIsBalooSearch;
+    bool mIsBalooSearch = false;
 };
 } // namespace MailCommon
 

src/search/widgethandler/tagrulewidgethandler.cpp

@@ -35,7 +35,7 @@ private Q_SLOTS:
     void onTagsFetched(KJob *);
 
 private:
-    QComboBox * mComboBox = nullptr;
+    QComboBox *mComboBox = nullptr;
 };
 
 FillTagComboJob::FillTagComboJob(QComboBox *combo, QObject *parent)

src/snippets/snippetvariabledialog.h

@@ -32,7 +32,7 @@ private:
     void writeConfig();
     void readConfig();
     void slotAccepted();
-    QString mVariableName;
+    const QString mVariableName;
     QMap<QString, QString> *mVariables = nullptr;
     KPIMTextEdit::PlainTextEditorWidget *mVariableValueText = nullptr;
     QCheckBox *mSaveVariable = nullptr;