Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: src/plugin/PluginFilter.cpp

Issue 5747779603267584: Issue #1234 - Rework strings in debug facility (Closed)
Patch Set: typo fix Created March 5, 2015, 12:43 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/plugin/PluginDebugMacros.h ('k') | src/plugin/PluginStdAfx.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/plugin/PluginFilter.cpp
===================================================================
--- a/src/plugin/PluginFilter.cpp
+++ b/src/plugin/PluginFilter.cpp
@@ -570,7 +570,7 @@
{
#ifdef ENABLE_DEBUG_RESULT
DEBUG_HIDE_EL(indent + "HideEl::Found (tag/id) filter:" + idIt->second.m_filterText)
- CPluginDebug::DebugResultHiding(tagCString, L"id:" + id, idIt->second.m_filterText);
+ CPluginDebug::DebugResultHiding(tag, L"id:" + ToWstring(id), ToWstring(idIt->second.m_filterText));
#endif
return true;
}
@@ -584,7 +584,7 @@
{
#ifdef ENABLE_DEBUG_RESULT
DEBUG_HIDE_EL(indent + "HideEl::Found (?/id) filter:" + idIt->second.m_filterText)
- CPluginDebug::DebugResultHiding(tagCString, L"id:" + id, idIt->second.m_filterText);
+ CPluginDebug::DebugResultHiding(tag, L"id:" + ToWstring(id), ToWstring(idIt->second.m_filterText));
#endif
return true;
}
@@ -607,7 +607,7 @@
{
#ifdef ENABLE_DEBUG_RESULT
DEBUG_HIDE_EL(indent + "HideEl::Found (tag/class) filter:" + classIt->second.m_filterText)
- CPluginDebug::DebugResultHiding(tagCString, L"class:" + className, classIt->second.m_filterText);
+ CPluginDebug::DebugResultHiding(tag, L"class:" + ToWstring(className), ToWstring(classIt->second.m_filterText));
#endif
return true;
}
@@ -620,8 +620,8 @@
if (classIt->second.IsMatchFilterElementHide(pEl))
{
#ifdef ENABLE_DEBUG_RESULT
- DEBUG_HIDE_EL(indent + "HideEl::Found (?/class) filter:" + classIt->second.m_filterText)
- CPluginDebug::DebugResultHiding(tagCString, "class:" + className, classIt->second.m_filterText);
+ DEBUG_HIDE_EL(indent + L"HideEl::Found (?/class) filter:" + ToWString(classIt->second.m_filterText));
+ CPluginDebug::DebugResultHiding(tag, L"class:" + ToWstring(className), ToWstring(classIt->second.m_filterText));
#endif
return true;
}
@@ -641,7 +641,7 @@
{
#ifdef ENABLE_DEBUG_RESULT
DEBUG_HIDE_EL(indent + "HideEl::Found (tag) filter:" + tagIt->second.m_filterText)
- CPluginDebug::DebugResultHiding(tagCString, "-", tagIt->second.m_filterText);
+ CPluginDebug::DebugResultHiding(tag, L"-", ToWstring(tagIt->second.m_filterText));
#endif
return true;
}
@@ -678,7 +678,7 @@
catch(...)
{
#ifdef ENABLE_DEBUG_RESULT
- CPluginDebug::DebugResult(L"Error loading hide filter: " + filter);
+ CPluginDebug::DebugResult(L"Error loading hide filter: " + ToWstring(filter));
#endif
}
}
@@ -719,11 +719,11 @@
std::wstring type = ToUtf16String(AdblockPlus::FilterEngine::ContentTypeToString(contentType));
if (result)
{
- CPluginDebug::DebugResultBlocking(ToCString(type), srcTrimmed, domain);
+ CPluginDebug::DebugResultBlocking(type, srcTrimmed, domain);
}
else
{
- CPluginDebug::DebugResultIgnoring(ToCString(type), srcTrimmed, domain);
+ CPluginDebug::DebugResultIgnoring(type, srcTrimmed, domain);
}
}
#endif
« no previous file with comments | « src/plugin/PluginDebugMacros.h ('k') | src/plugin/PluginStdAfx.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld