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

Unified Diff: test/FilterEngine.cpp

Issue 29421715: Noissue - fix regression in tests (Closed) Base URL: https://github.com/adblockplus/libadblockplus.git
Patch Set: Created April 25, 2017, 8:06 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/FilterEngine.cpp
diff --git a/test/FilterEngine.cpp b/test/FilterEngine.cpp
index 800a88411eba3c003371e371a925eb70847377c3..30dad91090f865a6b045e00c289b777073938bd7 100644
--- a/test/FilterEngine.cpp
+++ b/test/FilterEngine.cpp
@@ -942,8 +942,8 @@ namespace AA_ApiTest
auto subscriptions = filterEngine->GetListedSubscriptions();
for (auto& subscription : subscriptions)
{
- (subscription.IsAA() ? aaSubscription : otherSubscription) =
- std::unique_ptr<Subscription>(new Subscription(std::move(subscription)));
+ auto& dstSubscription = subscription.IsAA() ? aaSubscription : otherSubscription;
+ dstSubscription.reset(new Subscription(std::move(subscription)));
}
if (otherSubscriptionsNumber == 1u)
{
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld