- Use T && f() && to avoid moving temporaries.

- std::make_unique/make_shared

1 files changed, 2 insertions, 2 deletions

diff --git a/documentapi/src/tests/policies/testframe.cpp b/documentapi/src/tests/policies/testframe.cpp
index 973297a50d1..8cc13b116b6 100644
--- a/documentapi/src/tests/policies/testframe.cpp
+++ b/documentapi/src/tests/policies/testframe.cpp
@@ -111,8 +111,8 @@ TestFrame::testSelect(const std::vector<string> &expected)
     std::vector<mbus::RoutingNode*> selected;
     if (!select(selected, expected.size())) {
         LOG(error, "Failed to select recipients.");
-        for (auto & i : selected) {
-            LOG(error, "Selected: %s", i->getRoute().toString().c_str());
+        for (auto & node : selected) {
+            LOG(error, "Selected: %s", node->getRoute().toString().c_str());
         }
         return false;
     }