Revert "[test][clangd] Fix use-after-return after 72142fbac4"
authorVitaly Buka <vitalybuka@google.com>
Sun, 11 Sep 2022 18:44:38 +0000 (11:44 -0700)
committerVitaly Buka <vitalybuka@google.com>
Sun, 11 Sep 2022 18:46:58 +0000 (11:46 -0700)
Will try another fix.

This reverts commit c3c930d573656a825523b7112891bd97eec7b64f.

clang-tools-extra/clangd/unittests/TUSchedulerTests.cpp

index b2ab97e..4e95fcf 100644 (file)
@@ -1227,14 +1227,14 @@ TEST_F(TUSchedulerTests, IncluderCache) {
   auto GetFlags = [&](PathRef Header) {
     S.update(Header, getInputs(Header, ";"), WantDiagnostics::Yes);
     EXPECT_TRUE(S.blockUntilIdle(timeoutSeconds(10)));
-    auto Cmd = std::make_shared<tooling::CompileCommand>();
+    tooling::CompileCommand Cmd;
     S.runWithPreamble("GetFlags", Header, TUScheduler::StaleOrAbsent,
-                      [Cmd](llvm::Expected<InputsAndPreamble> Inputs) {
+                      [&](llvm::Expected<InputsAndPreamble> Inputs) {
                         ASSERT_FALSE(!Inputs) << Inputs.takeError();
-                        *Cmd = std::move(Inputs->Command);
+                        Cmd = std::move(Inputs->Command);
                       });
     EXPECT_TRUE(S.blockUntilIdle(timeoutSeconds(10)));
-    return Cmd->CommandLine;
+    return Cmd.CommandLine;
   };
 
   for (const auto &Path : {NoCmd, Unreliable, OK, NotIncluded})