First run page triggering

src/plugin/PluginUserSettings.cpp

 #include "PluginStdAfx.h"
 #include "PluginUserSettings.h"
 #include <algorithm>
 #include "PluginSettings.h"
 #include "PluginClient.h"
 #include "../shared/Dictionary.h"
 
 static const CString s_GetMessage = L"GetMessage";
 static const CString s_GetLanguageCount = L"GetLanguageCount";
 static const CString s_GetLanguageByIndex = L"GetLanguageByIndex";
@@ skipping 82 matching lines @@
 static CString sGetLanguage()
 {
   CPluginSettings* settings = CPluginSettings::GetInstance();
   return settings->GetSubscription();
 }
 
 
 CStringW sGetMessage(const CString& section, const CString& key)
 {
   Dictionary* dictionary = Dictionary::GetInstance();
-  return CString(dictionary->Lookup(std::string(CW2A(section)), std::string(CW2A(key))).c_str());

[Wladimir Palant, 2013/07/22 06:11:44]

Since you didn't address that comment in
http://codereview.adblockplus.org/11013110/ - how about changing CString into
CStringW here to match the return type?

+  return CStringW(dictionary->Lookup(std::string(CW2A(section)), std::string(CW2A(key))).c_str());
 }
 
 std::wstring sGetMessage(const std::string& section, const std::string& key)
 {
   Dictionary* dictionary = Dictionary::GetInstance();
   return dictionary->Lookup(section, key);
 }
 
 
 STDMETHODIMP CPluginUserSettings::Invoke(DISPID dispidMember, REFIID riid, LCID lcid, WORD wFlags, DISPPARAMS* pDispparams, VARIANT* pVarResult,
@@ skipping 190 matching lines @@
     {
       settings->RemoveWhiteListedDomain((BSTR)domain);
     }
   }
   else
     return DISP_E_MEMBERNOTFOUND;
 
   return S_OK;
 }