Use std::move rather than move

See https://reviews.llvm.org/D119670?id=408276 for the rationale.

Clang on macOS now has this on by default and pollutes builds with
warnings.

Author: ktf

Framework/test/testAggregatorRunner.cxx

@@ -73,7 +73,7 @@ BOOST_AUTO_TEST_CASE(test_aggregator_runner)
     { "qcConfiguration", VariantType::Dict, emptyDict(), { "Some dictionary configuration" } }
   };
   std::vector<std::unique_ptr<ParamRetriever>> retr;
-  std::unique_ptr<ConfigParamStore> store = make_unique<ConfigParamStore>(move(options), move(retr));
+  std::unique_ptr<ConfigParamStore> store = make_unique<ConfigParamStore>(std::move(options), std::move(retr));
   ConfigParamRegistry cfReg(std::move(store));
   ServiceRegistry sReg;
   InitContext initContext{ cfReg, sReg };
@@ -157,4 +157,4 @@ BOOST_AUTO_TEST_CASE(test_getDetector)
   auto checkEMC = std::make_shared<Aggregator>(config);
   aggregators.push_back(checkEMC);
   BOOST_CHECK_EQUAL(AggregatorRunner::getDetectorName(aggregators), "MANY");
-}
+}

Framework/test/testTaskInterface.cxx

@@ -41,7 +41,7 @@ o2::framework::ConfigParamRegistry createDummyRegistry()
 {
   using namespace o2::framework;
   std::unique_ptr<ParamRetriever> retriever;
-  ConfigParamRegistry registry(move(retriever));
+  ConfigParamRegistry registry(std::move(retriever));
 
   return registry;
 }

Framework/test/testTaskRunner.cxx

@@ -112,7 +112,7 @@ BOOST_AUTO_TEST_CASE(test_task_runner)
     { "qcConfiguration", VariantType::Dict, emptyDict(), { "Some dictionary configuration" } }
   };
   std::vector<std::unique_ptr<ParamRetriever>> retr;
-  std::unique_ptr<ConfigParamStore> store = make_unique<ConfigParamStore>(move(options), move(retr));
+  std::unique_ptr<ConfigParamStore> store = make_unique<ConfigParamStore>(std::move(options), std::move(retr));
   ConfigParamRegistry cfReg(std::move(store));
   ServiceRegistry sReg;
   InitContext initContext{ cfReg, sReg };