Unified Diff: src/plugin/PluginSettings.cpp
Issue 11427013:
Appear fast for IE (Closed)
Patch Set: Comments addressed
Created Aug. 7, 2013, 4:01 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« src/plugin/Config.h ('K')
|
« src/plugin/PluginSettings.h ('k')
|
src/plugin/PluginTabBase.cpp » ('j')
|
src/plugin/PluginTabBase.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: src/plugin/PluginSettings.cpp |
| =================================================================== |
| --- a/src/plugin/PluginSettings.cpp |
| +++ b/src/plugin/PluginSettings.cpp |
| @@ -93,10 +93,6 @@ |
| if (!s_instance) |
| { |
| s_instance = new CPluginSettings(); |
| -#ifdef USE_CONSOLE |
| - CONSOLE("Fetching Available Subscription\n"); |
| -#endif |
| - CPluginSettings::GetInstance()->m_subscriptions = CPluginClient::GetInstance()->FetchAvailableSubscriptions(); |
| } |
| instance = s_instance; |
| @@ -157,8 +153,10 @@ |
| } |
| -std::map<CString, CString> CPluginSettings::GetFilterLanguageTitleList() const |
| +std::map<CString, CString> CPluginSettings::GetFilterLanguageTitleList() |
| { |
| + m_subscriptions = CPluginClient::GetInstance()->FetchAvailableSubscriptions(); |
|
Wladimir Palant
2013/08/13 09:41:02
This should be a local variable, no point keeping
|
| + |
| std::map<CString, CString> filterList; |
| for (size_t i = 0; i < m_subscriptions.size(); i ++) |
| { |
« src/plugin/Config.h ('K')
|
« src/plugin/PluginSettings.h ('k')
|
src/plugin/PluginTabBase.cpp » ('j')
|
src/plugin/PluginTabBase.cpp » ('J')
