projects / nextcloud-desktop.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 8313d79)
Declare all QRegularExpressions statically
authorClaudio Cambra <claudio.cambra@nextcloud.com>
Mon, 19 Dec 2022 16:26:15 +0000 (17:26 +0100)
committerMatthieu Gallien <matthieu_gallien@yahoo.fr>
Tue, 20 Dec 2022 17:38:24 +0000 (18:38 +0100)
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
src/gui/creds/oauth.cpp patch | blob | history
src/gui/editlocallyjob.cpp patch | blob | history
src/gui/notificationconfirmjob.cpp patch | blob | history
src/gui/syncrunfilelog.cpp patch | blob | history
src/libsync/logger.cpp patch | blob | history
src/libsync/networkjobs.cpp patch | blob | history
src/libsync/propagatedownload.cpp patch | blob | history
src/libsync/theme.cpp patch | blob | history

diff --git a/src/gui/creds/oauth.cpp b/src/gui/creds/oauth.cpp
index 39534fa48bc185810ce2dfd18cc4522346b12198..04b0e62f865f520fa163826c2cc7216ec6b86d44 100644 (file)
--- a/src/gui/creds/oauth.cpp
+++ b/src/gui/creds/oauth.cpp
@@ -70,7 +70,7 @@ void OAuth::start()
                 QByteArray peek = socket->peek(qMin(socket->bytesAvailable(), 4000LL)); //The code should always be within the first 4K
                 if (peek.indexOf('\n') < 0)
                     return; // wait until we find a \n
-                const QRegularExpression rx("^GET /\\?code=([a-zA-Z0-9]+)[& ]"); // Match a  /?code=...  URL
+                static const QRegularExpression rx("^GET /\\?code=([a-zA-Z0-9]+)[& ]"); // Match a  /?code=...  URL
                 const auto rxMatch = rx.match(peek);
                 if (!rxMatch.hasMatch()) {
                     httpReplyAndClose(socket, "404 Not Found", "<html><head><title>404 Not Found</title></head><body><center><h1>404 Not Found</h1></center></body></html>");
diff --git a/src/gui/editlocallyjob.cpp b/src/gui/editlocallyjob.cpp
index 5e1b2837c1e72162e930fcbd45af401dc26e9a2b..fc18fddd4e18dfd3255a5a90fa5926eb47326be6 100644 (file)
--- a/src/gui/editlocallyjob.cpp
+++ b/src/gui/editlocallyjob.cpp
@@ -319,7 +319,7 @@ bool EditLocallyJob::isTokenValid(const QString &token)
 
     // Token is an alphanumeric string 128 chars long.
     // Ensure that is what we received and what we are sending to the server.
-    const QRegularExpression tokenRegex("^[a-zA-Z0-9]{128}$");
+    static const QRegularExpression tokenRegex("^[a-zA-Z0-9]{128}$");
     const auto regexMatch = tokenRegex.match(token);
 
     return regexMatch.hasMatch();
diff --git a/src/gui/notificationconfirmjob.cpp b/src/gui/notificationconfirmjob.cpp
index 09e147da3a7d855c62bfe12ebbf60a0158cfc654..8071aefddb4ef593b473cc0604d6e8ef66eca364 100644 (file)
--- a/src/gui/notificationconfirmjob.cpp
+++ b/src/gui/notificationconfirmjob.cpp
@@ -56,7 +56,7 @@ bool NotificationConfirmJob::finished()
     const QString replyStr = reply()->readAll();
 
     if (replyStr.contains("<?xml version=\"1.0\"?>")) {
-        const QRegularExpression rex("<statuscode>(\\d+)</statuscode>");
+        static const QRegularExpression rex("<statuscode>(\\d+)</statuscode>");
         const auto rexMatch = rex.match(replyStr);
         if (rexMatch.hasMatch()) {
             // this is a error message coming back from ocs.
diff --git a/src/gui/syncrunfilelog.cpp b/src/gui/syncrunfilelog.cpp
index 6f75f53cefbff4517e8a3e0bea2ec42382b4d458..f04636b4fc902f06393d97d27bb0d6127d4652de 100644 (file)
--- a/src/gui/syncrunfilelog.cpp
+++ b/src/gui/syncrunfilelog.cpp
@@ -104,7 +104,7 @@ void SyncRunFileLog::logItem(const SyncFileItem &item)
     }
     QString ts = QString::fromLatin1(item._responseTimeStamp);
     if (ts.length() > 6) {
-        const QRegularExpression rx(R"((\d\d:\d\d:\d\d))");
+        static const QRegularExpression rx(R"((\d\d:\d\d:\d\d))");
         const auto rxMatch = rx.match(ts);
         if (rxMatch.hasMatch()) {
             ts = rxMatch.captured(0);
diff --git a/src/libsync/logger.cpp b/src/libsync/logger.cpp
index 6087717486ca5317d30f776bd156f78d7448f82d..f6c95e375857621a9dcec66b05a1fc30bb00adeb 100644 (file)
--- a/src/libsync/logger.cpp
+++ b/src/libsync/logger.cpp
@@ -306,7 +306,7 @@ void Logger::enterNextLogFile()
         // Expire old log files and deal with conflicts
         QStringList files = dir.entryList(QStringList("*owncloud.log.*"), QDir::Files, QDir::Name) +
             dir.entryList(QStringList("*nextcloud.log.*"), QDir::Files, QDir::Name);
-        const QRegularExpression rx(QRegularExpression::anchoredPattern(R"(.*(next|own)cloud\.log\.(\d+).*)"));
+        static const QRegularExpression rx(QRegularExpression::anchoredPattern(R"(.*(next|own)cloud\.log\.(\d+).*)"));
         int maxNumber = -1;
         foreach (const QString &s, files) {
             if (_logExpire > 0) {
diff --git a/src/libsync/networkjobs.cpp b/src/libsync/networkjobs.cpp
index dd26a9550a06743f08f6fd193d3d8284989d65c1..19951674dbbe878a44528d576bf372a5bb384e21 100644 (file)
--- a/src/libsync/networkjobs.cpp
+++ b/src/libsync/networkjobs.cpp
@@ -840,7 +840,7 @@ bool JsonApiJob::finished()
 
     QString jsonStr = QString::fromUtf8(reply()->readAll());
     if (jsonStr.contains("<?xml version=\"1.0\"?>")) {
-        const QRegularExpression rex("<statuscode>(\\d+)</statuscode>");
+        static const QRegularExpression rex("<statuscode>(\\d+)</statuscode>");
         const auto rexMatch = rex.match(jsonStr);
         if (rexMatch.hasMatch()) {
             // this is a error message coming back from ocs.
@@ -850,7 +850,7 @@ bool JsonApiJob::finished()
         qCWarning(lcJsonApiJob) << "Nothing changed so nothing to retrieve - status code: " << httpStatusCode;
         statusCode = httpStatusCode;
     } else {
-        const QRegularExpression rex(R"("statuscode":(\d+))");
+        static const QRegularExpression rex(R"("statuscode":(\d+))");
         // example: "{"ocs":{"meta":{"status":"ok","statuscode":100,"message":null},"data":{"version":{"major":8,"minor":"... (504)
         const auto rxMatch = rex.match(jsonStr);
         if (rxMatch.hasMatch()) {
diff --git a/src/libsync/propagatedownload.cpp b/src/libsync/propagatedownload.cpp
index 0bb179b8168cdf27cdd3f536a50603e73b3defdc..b390cee5585f145d244ab9228dd69195460b4795 100644 (file)
--- a/src/libsync/propagatedownload.cpp
+++ b/src/libsync/propagatedownload.cpp
@@ -219,7 +219,7 @@ void GETFileJob::slotMetaDataChanged()
     qint64 start = 0;
     QByteArray ranges = reply()->rawHeader("Content-Range");
     if (!ranges.isEmpty()) {
-        const QRegularExpression rx("bytes (\\d+)-");
+        static const QRegularExpression rx("bytes (\\d+)-");
         const auto rxMatch = rx.match(ranges);
         if (rxMatch.hasMatch()) {
             start = rxMatch.captured(1).toLongLong();
diff --git a/src/libsync/theme.cpp b/src/libsync/theme.cpp
index e9d3206c999dcfceb36a6e8d6cd275847e176742..0a4b62820ebd4a5394a3678745e7f6eded658166 100644 (file)
--- a/src/libsync/theme.cpp
+++ b/src/libsync/theme.cpp
@@ -813,7 +813,8 @@ void Theme::replaceLinkColorStringBackgroundAware(QString &linkString)
 
 void Theme::replaceLinkColorString(QString &linkString, const QColor &newColor)
 {
-    linkString.replace(QRegularExpression("(<a href|<a style='color:#([a-zA-Z0-9]{6});' href)"), QString::fromLatin1("<a style='color:%1;' href").arg(newColor.name()));
+    static const QRegularExpression linkRegularExpression("(<a href|<a style='color:#([a-zA-Z0-9]{6});' href)");
+    linkString.replace(linkRegularExpression, QString::fromLatin1("<a style='color:%1;' href").arg(newColor.name()));
 }
 
 QIcon Theme::createColorAwareIcon(const QString &name, const QPalette &palette)
Unnamed repository; edit this file 'description' to name the repository.