Issue #1234 - std::wstring version of UnescapeUrl

src/plugin/PluginClientBase.h

 #ifndef _PLUGIN_CLIENT_BASE_H_
 #define _PLUGIN_CLIENT_BASE_H_
 
 
 #include "PluginTypedef.h"
 
 
 class CPluginClientFactory;
 
 
@@ skipping 58 matching lines @@
 
   static bool IsValidDomain(const CString& domain);
   static CString& UnescapeUrl(CString& url);
 
   static void LogPluginError(DWORD errorCode, int errorId, int errorSubid, const CString& description="", bool isAsync=false, DWORD dwProcessId=0, DWORD dwThreadId=0);
 
   static void PostPluginError(int errorId, int errorSubid, DWORD errorCode, const CString& errorDescription);
   static bool PopFirstPluginError(CPluginError& pluginError);
 };
 
+/**
+ * Wrapper around Microsoft API 'UrlUnescape'
+ *
+ * This function has modify-in-place semantics.
+ * This behavior matches that of the legacy version of this function declared above.
+ * At present, callers of this function have no code to handle error conditions that might arise here.
+ * Because there's no error handling, therefore, this masks failures in UrlUnescape.
+ */
+void UnescapeUrl(std::wstring& url);
 
 #endif // _PLUGIN_CLIENT_BASE_H_