Modernize OriginDialog

Use type safe connect syntax and remove one Q_FOREACH makro.
Make the reject button a non-member, but keep the OK button. I thought
about gettring rid of the latter, too, but deactivating OK when no match
is the most sensible way to go...

2 files changed, 5 insertions, 6 deletions

diff --git a/origindialog.cpp b/origindialog.cpp
index f885132..c2f5835 100644
--- a/origindialog.cpp
+++ b/origindialog.cpp
@@ -64,13 +64,13 @@ void OriginDialog::setupGui(){
     dstLayout->addWidget(mDstMd5, 3, 1);
     destGB->setLayout(dstLayout);
     mOk = new QPushButton(tr("Add"));
-    connect(mOk, SIGNAL(clicked()), this, SLOT(accept()));
-    mCancel = new QPushButton(tr("Cancel"));
-    connect(mCancel, SIGNAL(clicked()), this, SLOT(reject()));
+    connect(mOk, &QPushButton::clicked, this, &OriginDialog::accept);
+    QPushButton *cancelB = new QPushButton(tr("Cancel"));
+    connect(cancelB, &QPushButton::clicked, this, &OriginDialog::reject);
     QHBoxLayout *btnLayout = new QHBoxLayout;
     btnLayout->addStretch();
     btnLayout->addWidget(mOk);
-    btnLayout->addWidget(mCancel);
+    btnLayout->addWidget(cancelB);
     QVBoxLayout *mainLayout = new QVBoxLayout;
     mainLayout->addWidget(origGB);
     mainLayout->addWidget(destGB);
@@ -142,7 +142,7 @@ void OriginDialog::setDstData(const QString &absolutefn){
 }
 
 void OriginDialog::clear(){
-    foreach(QLineEdit *le, mLineEdits){
+    for(QLineEdit *le : mLineEdits){
         le->clear();
     }
 }
diff --git a/origindialog.h b/origindialog.h
index dd9cb3f..2380af6 100644
--- a/origindialog.h
+++ b/origindialog.h
@@ -35,7 +35,6 @@ class OriginDialog : public QDialog {
         QLineEdit *mDstMd5;
         QVector<QLineEdit*> mLineEdits;
         QPushButton *mOk;
-        QPushButton *mCancel;
         qint64 mSeconds;
         qint64 mSize;
         qint64 mBitrate;