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

Issue 29421562: Noissue - add copy and move constructors and assign operators for Filter (Closed)

Created:
April 25, 2017, 8:32 a.m. by sergei
Modified:
April 25, 2017, 12:07 p.m.
Reviewers:
hub
CC:
Felix Dahlke
Base URL:
https://github.com/adblockplus/libadblockplus.git
Visibility:
Public.

Description

Review: https://codereview.adblockplus.org/29421562

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -0 lines) Patch
M include/AdblockPlus/FilterEngine.h View 2 chunks +7 lines, -0 lines 0 comments Download
M src/FilterEngine.cpp View 1 chunk +22 lines, -0 lines 0 comments Download

Messages

Total messages: 2
sergei
April 25, 2017, 8:33 a.m. (2017-04-25 08:33:26 UTC) #1
hub
April 25, 2017, 12:03 p.m. (2017-04-25 12:03:26 UTC) #2
On 2017/04/25 08:33:26, sergei wrote:

LGTM

Powered by Google App Engine
This is Rietveld