]> granicus.if.org Git - clang/commitdiff
LexerTest memory leak fix-
authorErich Keane <erich.keane@intel.com>
Thu, 15 Jun 2017 18:34:47 +0000 (18:34 +0000)
committerErich Keane <erich.keane@intel.com>
Thu, 15 Jun 2017 18:34:47 +0000 (18:34 +0000)
A new LexerTest unittest introduced a memory leak.  This patch
uses a unique_ptr with a custom deleter to ensure it is properly
deleted.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@305491 91177308-0d34-0410-b5e6-96231b3b80d8

unittests/Lex/LexerTest.cpp

index 4bb355393f70ce1279d9e4492ff4b1b51da761b2..a887c22bd301e6120ff709a1dc3d897c4f6d2127 100644 (file)
@@ -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<MacroArgs, decltype(MacroArgsDeleter)> 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());