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

Unified Diff: compiled/subscription/Subscription.cpp

Issue 29426559: Issue 5137 - [emscripten] Added basic filter storage implementation (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore
Patch Set: Fixed bogus assert Created Aug. 31, 2017, 12:44 p.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 | « compiled/subscription/Subscription.h ('k') | compiled/subscription/UserDefinedSubscription.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: compiled/subscription/Subscription.cpp
===================================================================
--- a/compiled/subscription/Subscription.cpp
+++ b/compiled/subscription/Subscription.cpp
@@ -34,28 +34,28 @@ Subscription::Subscription(Type type, co
annotate_address(this, "Subscription");
}
Subscription::~Subscription()
{
knownSubscriptions.erase(mID);
}
-Filter* Subscription::FilterAt(unsigned index)
+Filter* Subscription::FilterAt(Subscription::Filters::size_type index)
{
if (index >= mFilters.size())
return nullptr;
FilterPtr result(mFilters[index]);
return result.release();
}
int Subscription::IndexOfFilter(Filter* filter)
{
- for (unsigned i = 0; i < mFilters.size(); i++)
+ for (Filters::size_type i = 0; i < mFilters.size(); i++)
if (mFilters[i] == filter)
return i;
return -1;
}
OwnedString Subscription::Serialize() const
{
OwnedString result(u"[Subscription]\nurl="_str);
« no previous file with comments | « compiled/subscription/Subscription.h ('k') | compiled/subscription/UserDefinedSubscription.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld