Use static QRegularExpressions as recommended by Clazy

This commit is contained in:
Cameron Gutman 2025-04-04 22:13:45 -05:00
parent e807a52cfa
commit 48bcd725c9

View File

@ -94,7 +94,7 @@ public:
bool getToggleOptionValue(QString name, bool defaultValue) const 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); QStringList options = optionNames().filter(re);
if (options.isEmpty()) { if (options.isEmpty()) {
return defaultValue; return defaultValue;
@ -113,7 +113,7 @@ public:
QPair<int,int> getResolutionOptionValue(QString name) const QPair<int,int> 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)); auto match = re.match(value(name));
if (!match.hasMatch()) { if (!match.hasMatch()) {
showError(QString("Invalid %1 format: %2").arg(name, value(name))); showError(QString("Invalid %1 format: %2").arg(name, value(name)));
@ -386,7 +386,7 @@ void StreamCommandLineParser::parse(const QStringList &args, StreamingPreference
parser.handleUnknownOptions(); parser.handleUnknownOptions();
// Resolve display's width and height // 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); QStringList resoOptions = parser.optionNames().filter(resolutionRexExp);
bool displaySet = !resoOptions.isEmpty(); bool displaySet = !resoOptions.isEmpty();
if (displaySet) { if (displaySet) {