diff --git a/src/gui/transferlistfilterswidget.cpp b/src/gui/transferlistfilterswidget.cpp index b3c80a010..448217cfa 100644 --- a/src/gui/transferlistfilterswidget.cpp +++ b/src/gui/transferlistfilterswidget.cpp @@ -66,18 +66,18 @@ namespace QString getScheme(const QString &tracker) { - const QUrl url {tracker}; - QString scheme = url.scheme(); - if (scheme.isEmpty()) - scheme = u"http"_qs; - return scheme; + const QString scheme = QUrl(tracker).scheme(); + return !scheme.isEmpty() ? scheme : u"http"_qs; } - QString getHost(const QString &tracker) + QString getHost(const QString &url) { // We want the domain + tld. Subdomains should be disregarded - const QUrl url {tracker}; - const QString host {url.host()}; + // If failed to parse the domain or IP address, original input should be returned + + const QString host = QUrl(url).host(); + if (host.isEmpty()) + return url; // host is in IP format if (!QHostAddress(host).isNull()) @@ -360,7 +360,6 @@ void StatusFilterWidget::torrentAboutToBeDeleted(BitTorrent::Torrent *const torr TrackerFiltersList::TrackerFiltersList(QWidget *parent, TransferListWidget *transferList, const bool downloadFavicon) : BaseFilterWidget(parent, transferList) - , m_totalTorrents(0) , m_downloadTrackerFavicon(downloadFavicon) { auto *allTrackers = new QListWidgetItem(this); @@ -509,10 +508,8 @@ void TrackerFiltersList::addItems(const QString &trackerURL, const QVector torrentIDs = m_trackers.value(host).torrents; - if (torrentIDs.empty()) - return; + QSet torrentIDs = m_trackers.value(host).torrents; torrentIDs.remove(id); QListWidgetItem *trackerItem = nullptr; diff --git a/src/gui/transferlistfilterswidget.h b/src/gui/transferlistfilterswidget.h index f911b2814..4745d6540 100644 --- a/src/gui/transferlistfilterswidget.h +++ b/src/gui/transferlistfilterswidget.h @@ -157,12 +157,12 @@ private: QListWidgetItem *item = nullptr; }; - QHash m_trackers; + QHash m_trackers; // QHash> m_errors; // QHash> m_warnings; // PathList m_iconPaths; - int m_totalTorrents; - bool m_downloadTrackerFavicon; + int m_totalTorrents = 0; + bool m_downloadTrackerFavicon = false; }; class TransferListFiltersWidget final : public QFrame