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

Unified Diff: include/AdblockPlus/FilterEngine.h

Issue 29421562: Noissue - add copy and move constructors and assign operators for Filter (Closed) Base URL: https://github.com/adblockplus/libadblockplus.git
Patch Set: Created April 25, 2017, 8:32 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 | « no previous file | src/FilterEngine.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include/AdblockPlus/FilterEngine.h
diff --git a/include/AdblockPlus/FilterEngine.h b/include/AdblockPlus/FilterEngine.h
index fe9cc42306f58ab9ad766b8d07f4a37fab71112f..de6fa666165069dc60c31f91b073ab233d8a5c94 100644
--- a/include/AdblockPlus/FilterEngine.h
+++ b/include/AdblockPlus/FilterEngine.h
@@ -39,7 +39,13 @@ namespace AdblockPlus
*/
class Filter : public JsValue
{
+ friend class FilterEngine;
public:
+ Filter(const Filter& src);
+ Filter(Filter&& src);
+ Filter& operator=(const Filter& src);
+ Filter& operator=(Filter&& src);
+
/**
* Filter types, see https://adblockplus.org/en/filters.
*/
@@ -71,6 +77,7 @@ namespace AdblockPlus
bool operator==(const Filter& filter) const;
+ protected:
/**
* Creates a wrapper for an existing JavaScript filter object.
* Normally you shouldn't call this directly, but use
« no previous file with comments | « no previous file | src/FilterEngine.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld