From 48bcd725c9163e3c50a5dc47ad2004cffb85af76 Mon Sep 17 00:00:00 2001 From: Cameron Gutman Date: Fri, 4 Apr 2025 22:13:45 -0500 Subject: [PATCH] Use static QRegularExpressions as recommended by Clazy --- app/cli/commandlineparser.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/app/cli/commandlineparser.cpp b/app/cli/commandlineparser.cpp index 0e50a75b..9c085ed7 100644 --- a/app/cli/commandlineparser.cpp +++ b/app/cli/commandlineparser.cpp @@ -94,7 +94,7 @@ public: bool getToggleOptionValue(QString name, bool defaultValue) const { - QRegularExpression re(QString("^(%1|no-%1)$").arg(name)); + static QRegularExpression re(QString("^(%1|no-%1)$").arg(name)); QStringList options = optionNames().filter(re); if (options.isEmpty()) { return defaultValue; @@ -113,7 +113,7 @@ public: QPair getResolutionOptionValue(QString name) const { - QRegularExpression re("^(\\d+)x(\\d+)$", QRegularExpression::CaseInsensitiveOption); + static QRegularExpression re("^(\\d+)x(\\d+)$", QRegularExpression::CaseInsensitiveOption); auto match = re.match(value(name)); if (!match.hasMatch()) { showError(QString("Invalid %1 format: %2").arg(name, value(name))); @@ -386,7 +386,7 @@ void StreamCommandLineParser::parse(const QStringList &args, StreamingPreference parser.handleUnknownOptions(); // Resolve display's width and height - QRegularExpression resolutionRexExp("^(720|1080|1440|4K|resolution)$"); + static QRegularExpression resolutionRexExp("^(720|1080|1440|4K|resolution)$"); QStringList resoOptions = parser.optionNames().filter(resolutionRexExp); bool displaySet = !resoOptions.isEmpty(); if (displaySet) {