[clang-tools-extra] r217162 - Plug a unit test memory leak.
    Benjamin Kramer 
    benny.kra at googlemail.com
       
    Thu Sep  4 08:15:27 PDT 2014
    
    
  
Author: d0k
Date: Thu Sep  4 10:15:27 2014
New Revision: 217162
URL: http://llvm.org/viewvc/llvm-project?rev=217162&view=rev
Log:
Plug a unit test memory leak.
MatchFinder.addMatcher doesn't take ownership.
Modified:
    clang-tools-extra/trunk/unittests/clang-modernize/TransformTest.cpp
Modified: clang-tools-extra/trunk/unittests/clang-modernize/TransformTest.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clang-modernize/TransformTest.cpp?rev=217162&r1=217161&r2=217162&view=diff
==============================================================================
--- clang-tools-extra/trunk/unittests/clang-modernize/TransformTest.cpp (original)
+++ clang-tools-extra/trunk/unittests/clang-modernize/TransformTest.cpp Thu Sep  4 10:15:27 2014
@@ -271,7 +271,8 @@ TEST(Transform, isFileModifiable) {
 
   DummyTransform T("dummy", Options);
   MatchFinder Finder;
-  Finder.addMatcher(varDecl().bind("decl"), new ModifiableCallback(T));
+  ModifiableCallback Callback(T);
+  Finder.addMatcher(varDecl().bind("decl"), &Callback);
   Tool.run(tooling::newFrontendActionFactory(&Finder).get());
 }
 
    
    
More information about the cfe-commits
mailing list