diff options
author | alex-z <blackslayer4@gmail.com> | 2021-10-29 17:56:29 +0300 |
---|---|---|
committer | allexzander (Rebase PR Action) <allexzander@users.noreply.github.com> | 2021-11-02 11:54:06 +0300 |
commit | c52718c104d69e054371e407e96ab1e132895772 (patch) | |
tree | 8942af619ba3ba71ffc974cb7cd4f055c2e3d34b /src/libsync | |
parent | e4eaf9d88d2f69c49e804ed993e7dbb7f0fbeefe (diff) |
Replace deprecated QRegExp with QRegularExpression.
Signed-off-by: alex-z <blackslayer4@gmail.com>
Diffstat (limited to 'src/libsync')
-rw-r--r-- | src/libsync/creds/credentialscommon.cpp | 1 | ||||
-rw-r--r-- | src/libsync/discovery.cpp | 2 | ||||
-rw-r--r-- | src/libsync/discoveryphase.h | 2 | ||||
-rw-r--r-- | src/libsync/logger.cpp | 8 | ||||
-rw-r--r-- | src/libsync/networkjobs.cpp | 14 | ||||
-rw-r--r-- | src/libsync/propagatedownload.cpp | 7 | ||||
-rw-r--r-- | src/libsync/syncengine.cpp | 4 |
7 files changed, 21 insertions, 17 deletions
diff --git a/src/libsync/creds/credentialscommon.cpp b/src/libsync/creds/credentialscommon.cpp index b56b8193d..edc17ff83 100644 --- a/src/libsync/creds/credentialscommon.cpp +++ b/src/libsync/creds/credentialscommon.cpp @@ -14,7 +14,6 @@ */ #include <QList> -#include <QRegExp> #include <QString> #include <QSslCertificate> #include <QSslConfiguration> diff --git a/src/libsync/discovery.cpp b/src/libsync/discovery.cpp index 8fd6506b1..3fefa7cde 100644 --- a/src/libsync/discovery.cpp +++ b/src/libsync/discovery.cpp @@ -247,7 +247,7 @@ bool ProcessDirectoryJob::handleExcluded(const QString &path, const QString &loc // FIXME: move to ExcludedFiles 's regexp ? bool isInvalidPattern = false; - if (excluded == CSYNC_NOT_EXCLUDED && !_discoveryData->_invalidFilenameRx.isEmpty()) { + if (excluded == CSYNC_NOT_EXCLUDED && !_discoveryData->_invalidFilenameRx.pattern().isEmpty()) { if (path.contains(_discoveryData->_invalidFilenameRx)) { excluded = CSYNC_FILE_EXCLUDE_INVALID_CHAR; isInvalidPattern = true; diff --git a/src/libsync/discoveryphase.h b/src/libsync/discoveryphase.h index a02807341..c894cfc2a 100644 --- a/src/libsync/discoveryphase.h +++ b/src/libsync/discoveryphase.h @@ -256,7 +256,7 @@ public: AccountPtr _account; SyncOptions _syncOptions; ExcludedFiles *_excludes; - QRegExp _invalidFilenameRx; // FIXME: maybe move in ExcludedFiles + QRegularExpression _invalidFilenameRx; // FIXME: maybe move in ExcludedFiles QStringList _serverBlacklistedFiles; // The blacklist from the capabilities bool _ignoreHiddenFiles = false; std::function<bool(const QString &)> _shouldDiscoverLocaly; diff --git a/src/libsync/logger.cpp b/src/libsync/logger.cpp index 5e956fbab..2a90c976b 100644 --- a/src/libsync/logger.cpp +++ b/src/libsync/logger.cpp @@ -17,6 +17,7 @@ #include "config.h" #include <QDir> +#include <QRegularExpression> #include <QStringList> #include <QtGlobal> #include <QTextCodec> @@ -281,7 +282,7 @@ void Logger::enterNextLogFile() // Expire old log files and deal with conflicts QStringList files = dir.entryList(QStringList("*owncloud.log.*"), QDir::Files, QDir::Name); - QRegExp rx(R"(.*owncloud\.log\.(\d+).*)"); + const QRegularExpression rx(QRegularExpression::anchoredPattern(R"(.*owncloud\.log\.(\d+).*)")); int maxNumber = -1; foreach (const QString &s, files) { if (_logExpire > 0) { @@ -290,8 +291,9 @@ void Logger::enterNextLogFile() dir.remove(s); } } - if (s.startsWith(newLogName) && rx.exactMatch(s)) { - maxNumber = qMax(maxNumber, rx.cap(1).toInt()); + const auto rxMatch = rx.match(s); + if (s.startsWith(newLogName) && rxMatch.hasMatch()) { + maxNumber = qMax(maxNumber, rxMatch.captured(1).toInt()); } } newLogName.append("." + QString::number(maxNumber + 1)); diff --git a/src/libsync/networkjobs.cpp b/src/libsync/networkjobs.cpp index 5bd60f8d4..f2ba661e4 100644 --- a/src/libsync/networkjobs.cpp +++ b/src/libsync/networkjobs.cpp @@ -892,19 +892,21 @@ bool JsonApiJob::finished() QString jsonStr = QString::fromUtf8(reply()->readAll()); if (jsonStr.contains("<?xml version=\"1.0\"?>")) { - QRegExp rex("<statuscode>(\\d+)</statuscode>"); - if (jsonStr.contains(rex)) { + const QRegularExpression rex("<statuscode>(\\d+)</statuscode>"); + const auto rexMatch = rex.match(jsonStr); + if (rexMatch.hasMatch()) { // this is a error message coming back from ocs. - statusCode = rex.cap(1).toInt(); + statusCode = rexMatch.captured(1).toInt(); } } else if(jsonStr.isEmpty() && httpStatusCode == notModifiedStatusCode){ qCWarning(lcJsonApiJob) << "Nothing changed so nothing to retrieve - status code: " << httpStatusCode; statusCode = httpStatusCode; } else { - QRegExp rex(R"("statuscode":(\d+))"); + const QRegularExpression rex(R"("statuscode":(\d+))"); // example: "{"ocs":{"meta":{"status":"ok","statuscode":100,"message":null},"data":{"version":{"major":8,"minor":"... (504) - if (jsonStr.contains(rex)) { - statusCode = rex.cap(1).toInt(); + const auto rxMatch = rex.match(jsonStr); + if (rxMatch.hasMatch()) { + statusCode = rxMatch.captured(1).toInt(); } } diff --git a/src/libsync/propagatedownload.cpp b/src/libsync/propagatedownload.cpp index e6b7383b9..10ccd97db 100644 --- a/src/libsync/propagatedownload.cpp +++ b/src/libsync/propagatedownload.cpp @@ -220,9 +220,10 @@ void GETFileJob::slotMetaDataChanged() qint64 start = 0; QByteArray ranges = reply()->rawHeader("Content-Range"); if (!ranges.isEmpty()) { - QRegExp rx("bytes (\\d+)-"); - if (rx.indexIn(ranges) >= 0) { - start = rx.cap(1).toLongLong(); + const QRegularExpression rx("bytes (\\d+)-"); + const auto rxMatch = rx.match(ranges); + if (rxMatch.hasMatch()) { + start = rxMatch.captured(1).toLongLong(); } } if (start != _resumeStart) { diff --git a/src/libsync/syncengine.cpp b/src/libsync/syncengine.cpp index 80a4c03cd..9813e63cb 100644 --- a/src/libsync/syncengine.cpp +++ b/src/libsync/syncengine.cpp @@ -579,7 +579,7 @@ void SyncEngine::startSync() invalidFilenamePattern = R"([\\:?*"<>|])"; } if (!invalidFilenamePattern.isEmpty()) - _discoveryPhase->_invalidFilenameRx = QRegExp(invalidFilenamePattern); + _discoveryPhase->_invalidFilenameRx = QRegularExpression(invalidFilenamePattern); _discoveryPhase->_serverBlacklistedFiles = _account->capabilities().blacklistedFiles(); _discoveryPhase->_ignoreHiddenFiles = ignoreHiddenFiles(); @@ -695,7 +695,7 @@ void SyncEngine::slotDiscoveryFinished() const QString script = qEnvironmentVariable("OWNCLOUD_POST_UPDATE_SCRIPT"); qCDebug(lcEngine) << "Post Update Script: " << script; - auto scriptArgs = script.split(QRegExp("\\s+"), Qt::SkipEmptyParts); + auto scriptArgs = script.split(QRegularExpression("\\s+"), Qt::SkipEmptyParts); if (scriptArgs.size() > 0) { const auto scriptExecutable = scriptArgs.takeFirst(); QProcess::execute(scriptExecutable, scriptArgs); |