[PATCH] Own/produce FrontendActionFactories by value, rather than new'd/unique_ptr ownership.

David Blaikie dblaikie at gmail.com
Thu Jun 26 09:57:15 PDT 2014


Hi klimek,

Based on a discussion on cfe-dev, simplify (and make more explicit, in places) the ownership semantics of FrontendActions and FrontendActionFactories using value semantics and std::unique_ptr in places.

http://reviews.llvm.org/D4313

Files:
  include/clang/Tooling/Refactoring.h
  include/clang/Tooling/Tooling.h
  lib/Tooling/Refactoring.cpp
  lib/Tooling/Tooling.cpp
  tools/clang-check/ClangCheck.cpp
  unittests/AST/ASTContextParentMapTest.cpp
  unittests/AST/DeclPrinterTest.cpp
  unittests/AST/DeclTest.cpp
  unittests/AST/EvaluateAsRValueTest.cpp
  unittests/AST/MatchVerifier.h
  unittests/AST/NamedDeclPrinterTest.cpp
  unittests/AST/StmtPrinterTest.cpp
  unittests/ASTMatchers/ASTMatchersTest.cpp
  unittests/ASTMatchers/ASTMatchersTest.h
  unittests/Sema/ExternalSemaSourceTest.cpp
  unittests/Tooling/CommentHandlerTest.cpp
  unittests/Tooling/RefactoringCallbacksTest.cpp
  unittests/Tooling/RefactoringTest.cpp
  unittests/Tooling/TestVisitor.h
  unittests/Tooling/ToolingTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4313.10893.patch
Type: text/x-patch
Size: 39282 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140626/4f3a4409/attachment.bin>


More information about the cfe-commits mailing list