]> granicus.if.org Git - clang/commit
[Tooling] [0/1] Refactor FrontendActionFactory::create() to return std::unique_ptr<>
authorRoman Lebedev <lebedev.ri@gmail.com>
Tue, 27 Feb 2018 15:19:20 +0000 (15:19 +0000)
committerRoman Lebedev <lebedev.ri@gmail.com>
Tue, 27 Feb 2018 15:19:20 +0000 (15:19 +0000)
commit9438256f7d6890d53e575269287b428086f48e70
tree1d02646971959478cde1e54d49c470c52c7ba41d
parent78ec87468cc30a36c657f0c57ed656fe401e40ef
[Tooling] [0/1] Refactor FrontendActionFactory::create() to return std::unique_ptr<>

Summary:
Noticed during review of D41102.

I'm not sure whether there are any principal reasons why it returns raw owning pointer,
or it is just a old code that was not updated post-C++11.

I'm not too sure what testing i should do, because `check-all` is not error clean here for some reason,
but it does not //appear// asif those failures are related to these changes.

This is clang part.
Clang-tools-extra part is D43780.

Reviewers: klimek, bkramer, alexfh, pcc

Reviewed By: alexfh

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D43779

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@326201 91177308-0d34-0410-b5e6-96231b3b80d8
14 files changed:
docs/LibTooling.rst
docs/RAVFrontendAction.rst
include/clang/Tooling/Tooling.h
lib/Tooling/Tooling.cpp
tools/clang-refactor/ClangRefactor.cpp
unittests/AST/EvaluateAsRValueTest.cpp
unittests/CrossTU/CrossTranslationUnitTest.cpp
unittests/Sema/CodeCompleteTest.cpp
unittests/Sema/ExternalSemaSourceTest.cpp
unittests/Tooling/CommentHandlerTest.cpp
unittests/Tooling/ExecutionTest.cpp
unittests/Tooling/RefactoringTest.cpp
unittests/Tooling/TestVisitor.h
unittests/Tooling/ToolingTest.cpp