Author: Kadir Cetinkaya Date: 2020-04-23T16:31:35+02:00 New Revision: 89cb5d558895706e053bc3af972aa5b15aa82863
URL: https://github.com/llvm/llvm-project/commit/89cb5d558895706e053bc3af972aa5b15aa82863 DIFF: https://github.com/llvm/llvm-project/commit/89cb5d558895706e053bc3af972aa5b15aa82863.diff LOG: [clangd] Delete remapped buffers in tests These buffers normally get freed after being used in a CompilerInstance. but tests don't make use of those, so we need to free them explicitly. Added: Modified: clang-tools-extra/clangd/unittests/PreambleTests.cpp Removed: ################################################################################ diff --git a/clang-tools-extra/clangd/unittests/PreambleTests.cpp b/clang-tools-extra/clangd/unittests/PreambleTests.cpp index 6382285b63fc..5ff7ebc27e09 100644 --- a/clang-tools-extra/clangd/unittests/PreambleTests.cpp +++ b/clang-tools-extra/clangd/unittests/PreambleTests.cpp @@ -91,6 +91,8 @@ TEST(PreamblePatchTest, IncludeParsing) { PreamblePatch::create(FileName, PI, *EmptyPreamble).apply(*CI); EXPECT_THAT(CI->getPreprocessorOpts().RemappedFileBuffers, Contains(Pair(_, HasContents(ExpectedBuffer)))); + for (const auto &RB : CI->getPreprocessorOpts().RemappedFileBuffers) + delete RB.second; } } @@ -120,6 +122,8 @@ TEST(PreamblePatchTest, ContainsNewIncludes) { PreamblePatch::create(FileName, PI, *FullPreamble).apply(*CI); EXPECT_THAT(CI->getPreprocessorOpts().RemappedFileBuffers, Contains(Pair(_, HasContents(Patch)))); + for (const auto &RB : CI->getPreprocessorOpts().RemappedFileBuffers) + delete RB.second; } } // namespace _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
