Changeset View
Changeset View
Standalone View
Standalone View
src/providermanager.cpp
Show First 20 Lines • Show All 97 Lines • ▼ Show 20 Line(s) | |||||
98 | void ProviderManager::clear() | 98 | void ProviderManager::clear() | ||
99 | { | 99 | { | ||
100 | d->m_providerTargets.clear(); | 100 | d->m_providerTargets.clear(); | ||
101 | d->m_providers.clear(); | 101 | d->m_providers.clear(); | ||
102 | } | 102 | } | ||
103 | 103 | | |||
104 | void ProviderManager::slotLoadDefaultProvidersInternal() | 104 | void ProviderManager::slotLoadDefaultProvidersInternal() | ||
105 | { | 105 | { | ||
106 | foreach (const QUrl &url, d->m_internals->getDefaultProviderFiles()) { | 106 | const auto providerFiles = d->m_internals->getDefaultProviderFiles(); | ||
107 | for (const QUrl &url : providerFiles) { | ||||
dfaure: the variable could be better named, like providerFiles or just files
(these are files, not… | |||||
107 | addProviderFile(url); | 108 | addProviderFile(url); | ||
108 | } | 109 | } | ||
109 | if (d->m_downloads.isEmpty()) { | 110 | if (d->m_downloads.isEmpty()) { | ||
110 | emit defaultProvidersLoaded(); | 111 | emit defaultProvidersLoaded(); | ||
111 | } | 112 | } | ||
112 | } | 113 | } | ||
113 | 114 | | |||
114 | QList<QUrl> ProviderManager::defaultProviderFiles() | 115 | QList<QUrl> ProviderManager::defaultProviderFiles() | ||
▲ Show 20 Lines • Show All 211 Lines • Show Last 20 Lines |
the variable could be better named, like providerFiles or just files
(these are files, not providers)