Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: src/plugin/PluginSettings.cpp

Issue 11612008: Bunch of tiny fixes (Closed)
Patch Set: Created Sept. 5, 2013, 10:38 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/plugin/PluginSettings.cpp
===================================================================
--- a/src/plugin/PluginSettings.cpp
+++ b/src/plugin/PluginSettings.cpp
@@ -153,14 +153,14 @@
}
Oleksandr 2013/09/05 22:47:40 http://codereview.adblockplus.org/11427013/diff/80
-std::map<CString, CString> CPluginSettings::GetFilterLanguageTitleList()
+std::map<CString, CString> CPluginSettings::GetFilterLanguageTitleList() const
{
- m_subscriptions = CPluginClient::GetInstance()->FetchAvailableSubscriptions();
+ std::vector<SubscriptionDescription> subscriptions = CPluginClient::GetInstance()->FetchAvailableSubscriptions();
std::map<CString, CString> filterList;
- for (size_t i = 0; i < m_subscriptions.size(); i ++)
+ for (size_t i = 0; i < subscriptions.size(); i ++)
{
- SubscriptionDescription it = m_subscriptions[i];
+ SubscriptionDescription it = subscriptions[i];
filterList.insert(std::make_pair(CString(it.url.c_str()), CString(it.title.c_str())));
}
return filterList;

Powered by Google App Engine
This is Rietveld