From 9ca45e08274d1880ae2b7f3db5abc3b2bfe169aa Mon Sep 17 00:00:00 2001 From: Zentino <52699319+zent1n0@users.noreply.github.com> Date: Wed, 18 Dec 2024 08:58:58 +0800 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Chocobo1 --- src/gui/rss/rsswidget.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gui/rss/rsswidget.cpp b/src/gui/rss/rsswidget.cpp index 4dea9d73c..aef2774c7 100644 --- a/src/gui/rss/rsswidget.cpp +++ b/src/gui/rss/rsswidget.cpp @@ -63,7 +63,7 @@ namespace const QRegularExpression rx {uR"(((]*?href|]*?src)\s*=\s*["'])((https?|ftp):)?(\/\/[^\/]*)?(\/?[^\/"].*?)(["']))"_s , QRegularExpression::CaseInsensitiveOption}; - const QString normalizedBaseUrl = baseUrl.endsWith(u'/') ? baseUrl : baseUrl + u'/'; + const QString normalizedBaseUrl = baseUrl.endsWith(u'/') ? baseUrl : (baseUrl + u'/'); const QUrl url {normalizedBaseUrl}; const QString defaultScheme = url.scheme(); QRegularExpressionMatchIterator iter = rx.globalMatch(html); @@ -87,7 +87,7 @@ namespace relativePath = relativePath.mid(1); const QString absoluteUrl = !host.isEmpty() - ? QString(defaultScheme + u":" + host) : QString(normalizedBaseUrl + relativePath); + ? QString(defaultScheme + u':' + host) : (normalizedBaseUrl + relativePath); const QString fullMatch = match.captured(0); const QString prefix = match.captured(1); const QString suffix = match.captured(7);