diff options
Diffstat (limited to '')
| -rw-r--r-- | lang/qt/src/qgpgmekeyformailboxjob.cpp | 8 | ||||
| -rw-r--r-- | lang/qt/src/qgpgmekeylistjob.cpp | 2 | ||||
| -rw-r--r-- | lang/qt/src/qgpgmenewcryptoconfig.cpp | 50 | ||||
| -rw-r--r-- | lang/qt/src/qgpgmesignencryptjob.cpp | 6 | ||||
| -rw-r--r-- | lang/qt/src/qgpgmesignjob.cpp | 6 | ||||
| -rw-r--r-- | lang/qt/src/threadedjobmixin.cpp | 2 | 
6 files changed, 41 insertions, 33 deletions
diff --git a/lang/qt/src/qgpgmekeyformailboxjob.cpp b/lang/qt/src/qgpgmekeyformailboxjob.cpp index b6ab3db6..8a6651ba 100644 --- a/lang/qt/src/qgpgmekeyformailboxjob.cpp +++ b/lang/qt/src/qgpgmekeyformailboxjob.cpp @@ -86,12 +86,12 @@ static QGpgMEKeyForMailboxJob::result_type do_work(Context *ctx, const QString &      // See: https://bugs.gnupg.org/gnupg/issue2359      Key keyC;      UserID uidC; -    Q_FOREACH (const Key k, keys) { +    for (const Key &k : keys) {          if (canEncrypt && !k.canEncrypt()) {              continue;          }          /* First get the uid that matches the mailbox */ -        Q_FOREACH (const UserID u, k.userIDs()) { +        for (const UserID &u : k.userIDs()) {              if (QString::fromUtf8(u.email()).toLower() == mailbox.toLower()) {                  if (uidC.isNull()) {                      keyC = k; @@ -103,13 +103,13 @@ static QGpgMEKeyForMailboxJob::result_type do_work(Context *ctx, const QString &                  } else if (uidC.validity() == u.validity() && uidIsOk(u)) {                      /* Both are the same check which one is newer. */                      time_t oldTime = 0; -                    Q_FOREACH (const Subkey s, keyC.subkeys()) { +                    for (const Subkey &s : keyC.subkeys()) {                          if ((canEncrypt && s.canEncrypt()) && subkeyIsOk(s)) {                              oldTime = s.creationTime();                          }                      }                      time_t newTime = 0; -                    Q_FOREACH (const Subkey s, k.subkeys()) { +                    for (const Subkey &s : k.subkeys()) {                          if ((canEncrypt && s.canEncrypt()) && subkeyIsOk(s)) {                              newTime = s.creationTime();                          } diff --git a/lang/qt/src/qgpgmekeylistjob.cpp b/lang/qt/src/qgpgmekeylistjob.cpp index 295e612d..706d2575 100644 --- a/lang/qt/src/qgpgmekeylistjob.cpp +++ b/lang/qt/src/qgpgmekeylistjob.cpp @@ -152,7 +152,7 @@ KeyListResult QGpgMEKeyListJob::exec(const QStringList &patterns, bool secretOnl  void QGpgMEKeyListJob::resultHook(const result_type &tuple)  {      mResult = std::get<0>(tuple); -    Q_FOREACH (const Key &key, std::get<1>(tuple)) { +    for (const Key &key : std::get<1>(tuple)) {          Q_EMIT nextKey(key);      }  } diff --git a/lang/qt/src/qgpgmenewcryptoconfig.cpp b/lang/qt/src/qgpgmenewcryptoconfig.cpp index 94e3fb2b..15d50b4a 100644 --- a/lang/qt/src/qgpgmenewcryptoconfig.cpp +++ b/lang/qt/src/qgpgmenewcryptoconfig.cpp @@ -97,7 +97,7 @@ void QGpgMENewCryptoConfig::reloadConfiguration(bool)          KMessageBox::error(0, wmsg);      }  #endif -    Q_FOREACH(const Component & c, components) { +    for (const Component &c : components) {          const std::shared_ptr<QGpgMENewCryptoConfigComponent> comp(new QGpgMENewCryptoConfigComponent);          comp->setComponent(c);          m_componentsByName[ comp->name() ] = comp; @@ -127,8 +127,9 @@ QGpgMENewCryptoConfigComponent *QGpgMENewCryptoConfig::component(const QString &  void QGpgMENewCryptoConfig::sync(bool runtime)  { -    Q_FOREACH(const std::shared_ptr<QGpgMENewCryptoConfigComponent> &c, m_componentsByName) -    c->sync(runtime); +    for (const std::shared_ptr<QGpgMENewCryptoConfigComponent> &c : qAsConst(m_componentsByName)) { +        c->sync(runtime); +    }  }  void QGpgMENewCryptoConfig::clear() @@ -156,19 +157,20 @@ void QGpgMENewCryptoConfigComponent::setComponent(const Component &component)      std::shared_ptr<QGpgMENewCryptoConfigGroup> group;      const std::vector<Option> options = m_component.options(); -    Q_FOREACH(const Option & o, options) -    if (o.flags() & Group) { -        if (group) { -            m_groupsByName[group->name()] = group; +    for (const Option &o : options) { +        if (o.flags() & Group) { +            if (group) { +                m_groupsByName[group->name()] = group; +            } +            group.reset(new QGpgMENewCryptoConfigGroup(shared_from_this(), o)); +        } else if (group) { +            const std::shared_ptr<QGpgMENewCryptoConfigEntry> entry(new QGpgMENewCryptoConfigEntry(group, o)); +            const QString name = entry->name(); +            group->m_entryNames.push_back(name); +            group->m_entriesByName[name] = entry; +        } else { +            qCWarning(QGPGME_LOG) << "found no group for entry" << o.name() << "of component" << name();          } -        group.reset(new QGpgMENewCryptoConfigGroup(shared_from_this(), o)); -    } else if (group) { -        const std::shared_ptr<QGpgMENewCryptoConfigEntry> entry(new QGpgMENewCryptoConfigEntry(group, o)); -        const QString name = entry->name(); -        group->m_entryNames.push_back(name); -        group->m_entriesByName[name] = entry; -    } else { -        qCWarning(QGPGME_LOG) << "found no group for entry" << o.name() << "of component" << name();      }      if (group) {          m_groupsByName[group->name()] = group; @@ -578,13 +580,14 @@ QList<QUrl> QGpgMENewCryptoConfigEntry::urlValueList() const      const Argument arg = m_option.currentValue();      const std::vector<const char *> values = arg.stringValues();      QList<QUrl> ret; -    Q_FOREACH(const char *value, values) -    if (type == FilenameType) { -        QUrl url; -        url.setPath(QFile::decodeName(value)); -        ret << url; -    } else { -        ret << parseURL(type, QString::fromUtf8(value)); +    for (const char *value : values) { +        if (type == FilenameType) { +            QUrl url; +            url.setPath(QFile::decodeName(value)); +            ret << url; +        } else { +            ret << parseURL(type, QString::fromUtf8(value)); +        }      }      return ret;  } @@ -678,12 +681,13 @@ void QGpgMENewCryptoConfigEntry::setURLValueList(const QList<QUrl> &urls)      Q_ASSERT(isList());      std::vector<std::string> values;      values.reserve(urls.size()); -    Q_FOREACH (const QUrl &url, urls) +    for (const QUrl &url : urls) {          if (type == FilenameType) {              values.push_back(QFile::encodeName(url.path()).constData());          } else {              values.push_back(splitURL(type, url).toUtf8().constData());          } +    }      const auto err = m_option.setNewValue(m_option.createStringListArgument(values));      if (err) {          qCWarning(QGPGME_LOG) << "setURLValueList: failed to set new value:" << err; diff --git a/lang/qt/src/qgpgmesignencryptjob.cpp b/lang/qt/src/qgpgmesignencryptjob.cpp index c2ced590..55513c9f 100644 --- a/lang/qt/src/qgpgmesignencryptjob.cpp +++ b/lang/qt/src/qgpgmesignencryptjob.cpp @@ -124,11 +124,13 @@ static QGpgMESignEncryptJob::result_type sign_encrypt(Context *ctx, QThread *thr      }      ctx->clearSigningKeys(); -    Q_FOREACH (const Key &signer, signers) -        if (!signer.isNull()) +    for (const Key &signer : signers) { +        if (!signer.isNull()) {              if (const Error err = ctx->addSigningKey(signer)) {                  return std::make_tuple(SigningResult(err), EncryptionResult(), QByteArray(), QString(), Error());              } +        } +    }      if (!cipherText) {          QGpgME::QByteArrayDataProvider out; diff --git a/lang/qt/src/qgpgmesignjob.cpp b/lang/qt/src/qgpgmesignjob.cpp index a5e4f0f2..fff242e9 100644 --- a/lang/qt/src/qgpgmesignjob.cpp +++ b/lang/qt/src/qgpgmesignjob.cpp @@ -87,11 +87,13 @@ static QGpgMESignJob::result_type sign(Context *ctx, QThread *thread,      }      ctx->clearSigningKeys(); -    Q_FOREACH (const Key &signer, signers) -        if (!signer.isNull()) +    for (const Key &signer : signers) { +        if (!signer.isNull()) {              if (const Error err = ctx->addSigningKey(signer)) {                  return std::make_tuple(SigningResult(err), QByteArray(), QString(), Error());              } +        } +    }      if (!signature) {          QGpgME::QByteArrayDataProvider out; diff --git a/lang/qt/src/threadedjobmixin.cpp b/lang/qt/src/threadedjobmixin.cpp index 92e63849..b7963c39 100644 --- a/lang/qt/src/threadedjobmixin.cpp +++ b/lang/qt/src/threadedjobmixin.cpp @@ -108,7 +108,7 @@ QString _detail::audit_log_as_html(Context *ctx, GpgME::Error &err)  static QList<QByteArray> from_sl(const QStringList &sl)  {      QList<QByteArray> result; -    Q_FOREACH (const QString &str, sl) { +    for (const QString &str : sl) {          result.append(str.toUtf8());      }  | 
