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

Unified Diff: compiled/String.cpp

Issue 29556737: Issue 5141 - Convert filter match to C++ (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Some more cleanup Created Sept. 29, 2017, 4:12 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
Index: compiled/String.cpp
===================================================================
new file mode 100644
--- /dev/null
+++ b/compiled/String.cpp
@@ -0,0 +1,32 @@
+/*
+ * This file is part of Adblock Plus <https://adblockplus.org/>,
+ * Copyright (C) 2006-present 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
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+
+#include "library.h"
+#include "String.h"
+
+OwnedString String::substr(size_type pos, size_type len) const
+{
+ if (len == npos)
+ len = length() - pos;
+ return OwnedString(&mBuf[pos], len);
sergei 2017/10/02 12:02:33 Will it be safer to return DependentString(*this,
hub 2017/10/06 13:49:15 Safer, not sure, but yes it seems we can.
+}
+
+bool String::match(int id, ReMatchResults* results) const
+{
+ return MatchRegExp(id, *this, results);
+}

Powered by Google App Engine
This is Rietveld