Issue 1103 - Migrate Simple Adblock users

src/plugin/PluginUserSettings.cpp

Index: src/plugin/PluginUserSettings.cpp
===================================================================
--- a/src/plugin/PluginUserSettings.cpp
+++ b/src/plugin/PluginUserSettings.cpp
@@ -41,6 +41,7 @@
     dispatchID_IsAcceptableAdsEnabled,
     dispatchID_SetAcceptableAdsEnabled,
     dispatchID_IsUpdate,
+    dispatchID_GetConversion
   };
 
   /**
@@ -66,6 +67,7 @@
       m.emplace(L"IsAcceptableAdsEnabled", dispatchID_IsAcceptableAdsEnabled);
       m.emplace(L"SetAcceptableAdsEnabled", dispatchID_SetAcceptableAdsEnabled);
       m.emplace(L"IsUpdate", dispatchID_IsUpdate);
+      m.emplace(L"GetConversion", dispatchID_GetConversion);
     }
     catch(...)
     {
@@ -442,6 +444,19 @@
         }
       }
       break;
+    case dispatchID_GetConversion:
+      {
+        if (pDispparams->cArgs != 0)
+        {
+          return DISP_E_BADPARAMCOUNT;
+        }
+        if (pVarResult)
+        {
+          pVarResult->vt = VT_BSTR;
+          pVarResult->bstrVal = CComBSTR(CPluginClient::GetInstance()->GetPref(std::wstring(L"convertedFrom"), std::wstring(L"")).c_str());

[sergei, 2015/04/28 14:41:15]

Is it necessary to explicitly construct std::wstring here?

[Eric, 2015/05/17 00:29:25]

It's not. There's an explicit pass through function for second argument
'wchar_t*' that invokes the 'std::wstring' version.

+        }
+      }
+      break;
     default:
       return DISP_E_MEMBERNOTFOUND;
       break;