From cb7af85b902eafedd180adde6c614ff9dfa09905 Mon Sep 17 00:00:00 2001 From: Erich Keane Date: Thu, 15 Jun 2017 18:34:47 +0000 Subject: [PATCH] LexerTest memory leak fix- A new LexerTest unittest introduced a memory leak. This patch uses a unique_ptr with a custom deleter to ensure it is properly deleted. llvm-svn: 305491 --- clang/unittests/Lex/LexerTest.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/clang/unittests/Lex/LexerTest.cpp b/clang/unittests/Lex/LexerTest.cpp index 4bb355393f70..a887c22bd301 100644 --- a/clang/unittests/Lex/LexerTest.cpp +++ b/clang/unittests/Lex/LexerTest.cpp @@ -402,7 +402,9 @@ TEST_F(LexerTest, DontOverallocateStringifyArgs) { ArgTokens.push_back(tok); } - MacroArgs *MA = MacroArgs::create(MI, ArgTokens, false, *PP); + auto MacroArgsDeleter = [&PP](MacroArgs *M) { M->destroy(*PP); }; + std::unique_ptr MA( + MacroArgs::create(MI, ArgTokens, false, *PP), MacroArgsDeleter); Token Result = MA->getStringifiedArgument(0, *PP, {}, {}); EXPECT_EQ(tok::string_literal, Result.getKind()); EXPECT_STREQ("\"\\\"StrArg\\\"\"", Result.getLiteralData());