Issue 1115 - Some yahoo page not correctly shown on IE8 when ABP enabled

src/shared/Utils.cpp

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 63 matching lines @@
   if (length == 0)
     return std::wstring();
 
   int utf16StringLength = MultiByteToWideChar(CP_UTF8, 0, str.c_str(), length, NULL, 0);
   if (utf16StringLength == 0)
     throw std::runtime_error("ToUTF16String failed. Can't determine the length of the buffer needed.");
 
   std::wstring utf16String(utf16StringLength, L'\0');
   MultiByteToWideChar(CP_UTF8, 0, str.c_str(), length, &utf16String[0], utf16StringLength);
   return utf16String;
+}
+
+std::vector<std::wstring> ToUtf16Strings(const std::vector<std::string>& values)
+{
+  std::vector<std::wstring> result;
+  result.reserve(values.size());
+  transform(values.begin(), values.end(), back_inserter(result), ToUtf16String);
+  return result;
 }
 
 std::wstring GetDllDir()
 {
   std::vector<WCHAR> path(MAX_PATH);
   int length = GetModuleFileNameW((HINSTANCE)&__ImageBase, &path[0], static_cast<DWORD>(path.size()));
 
   while (length == path.size())
   {
     // Buffer too small, double buffer size
@@ skipping 72 matching lines @@
     return L"";
   }
   auto queryStringBeginsAt = questionSignPos + 1;
   auto endQueryStringPos = url.find(L'#', queryStringBeginsAt);
   if (endQueryStringPos == std::wstring::npos)
   {
     endQueryStringPos = url.length();
   }
   return url.substr(queryStringBeginsAt, endQueryStringPos - queryStringBeginsAt);
 }
-
-void ForEachToken(const std::wstring& value, wchar_t delimiter,
-                 const std::function<bool(std::wstring::const_iterator begin, std::wstring::const_iterator end)>& tokenHandler)

[Eric, 2015/02/13 16:33:43]

You might change "tokenHandler" to "fn" here. Or at least "tokenFn".

This is not a strong opinion.

-{
-  if (value.empty())
-  {
-    return;
-  }
-  std::wstring::size_type delimiterPos = value.find(delimiter);
-  std::wstring::size_type prevDelimiterPos = 0;
-  while(delimiterPos != std::wstring::npos)
-  {
-    if (!tokenHandler(value.begin() + prevDelimiterPos, value.begin() + delimiterPos))
-    {
-      return;
-    }
-    prevDelimiterPos = delimiterPos;
-    ++prevDelimiterPos; // skip the delimiter
-    delimiterPos = value.find(delimiter, prevDelimiterPos);
-  }
-  tokenHandler(value.begin() + prevDelimiterPos, value.end());
-}
-
-void ForEachQueryStringParameter(const std::wstring& queryString,
-                                 const std::function<bool(const std::wstring& name, const std::wstring& value)>& parameterHandler)

[Eric, 2015/02/13 16:33:43]

See comment above.

-{
-  if (queryString.empty())
-  {
-    return;
-  }
-  ForEachToken(queryString, L'&',
-    [&parameterHandler](std::wstring::const_iterator begin,
-                        std::wstring::const_iterator end)->bool
-    {
-      auto assignSignIt = std::find(begin, end, L'=');
-      auto valueBeginsAt = assignSignIt != end ? assignSignIt + 1 : assignSignIt;
-      return parameterHandler(std::wstring(begin, assignSignIt), std::wstring(valueBeginsAt, end));
-    });
-}