diff --git a/folly/fibers/test/FibersTest.cpp b/folly/fibers/test/FibersTest.cpp index 0da0e4d4e27..c3c83b0aa30 100644 --- a/folly/fibers/test/FibersTest.cpp +++ b/folly/fibers/test/FibersTest.cpp @@ -1494,8 +1494,8 @@ TEST(FiberManager, remoteFutureTest) { auto f1 = fiberManager.addTaskFuture([&]() { return testValue1; }); auto f2 = fiberManager.addTaskRemoteFuture([&]() { return testValue2; }); loopController.loop([&]() { loopController.stop(); }); - auto v1 = f1.get(); - auto v2 = f2.get(); + auto v1 = std::move(f1).get(); + auto v2 = std::move(f2).get(); EXPECT_EQ(v1, testValue1); EXPECT_EQ(v2, testValue2); diff --git a/folly/futures/test/Benchmark.cpp b/folly/futures/test/Benchmark.cpp index fcc4a35398e..9e09f6d8116 100644 --- a/folly/futures/test/Benchmark.cpp +++ b/folly/futures/test/Benchmark.cpp @@ -319,7 +319,7 @@ BENCHMARK(lvalue_get) { Optional> future; future = makeFuture(Bulky("Hello")); suspender.dismissing([&] { - std::string message = future.value().get().message(); + std::string message = std::move(future.value()).get().message(); doNotOptimizeAway(message); }); } diff --git a/folly/logging/test/AsyncFileWriterTest.cpp b/folly/logging/test/AsyncFileWriterTest.cpp index 46f7bc05661..b186a6f9c4d 100644 --- a/folly/logging/test/AsyncFileWriterTest.cpp +++ b/folly/logging/test/AsyncFileWriterTest.cpp @@ -256,7 +256,7 @@ TEST(AsyncFileWriter, flush) { readFull(readPipe.fd(), buf.data(), buf.size()); // Make sure flush completes successfully now - future.get(10ms); + std::move(future).get(10ms); } // A large-ish message suffix, just to consume space and help fill up