diff options
| author | George Karpenkov <ekarpenkov@apple.com> | 2018-09-21 20:36:21 +0000 |
|---|---|---|
| committer | George Karpenkov <ekarpenkov@apple.com> | 2018-09-21 20:36:21 +0000 |
| commit | 6babf2ae169c87b3e7f8c25582b98927c0c07336 (patch) | |
| tree | 7ab227a6661e478f2a421fe6891ab8cac8b228fb /clang | |
| parent | a31c224bb4fb426a147db7a9b49d47917a2280ad (diff) | |
| download | bcm5719-llvm-6babf2ae169c87b3e7f8c25582b98927c0c07336.tar.gz bcm5719-llvm-6babf2ae169c87b3e7f8c25582b98927c0c07336.zip | |
[analyzer] [NFC] Prefer make_unique over "new"
Differential Revision: https://reviews.llvm.org/D52336
llvm-svn: 342767
Diffstat (limited to 'clang')
| -rw-r--r-- | clang/lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountChecker.cpp | 25 |
1 files changed, 11 insertions, 14 deletions
diff --git a/clang/lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountChecker.cpp index 351816ae208..de113bbd057 100644 --- a/clang/lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountChecker.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountChecker.cpp @@ -750,9 +750,8 @@ void RetainCountChecker::processNonLeakError(ProgramStateRef St, } assert(BT); - auto report = std::unique_ptr<BugReport>( - new CFRefReport(*BT, C.getASTContext().getLangOpts(), - SummaryLog, N, Sym)); + auto report = llvm::make_unique<CFRefReport>( + *BT, C.getASTContext().getLangOpts(), SummaryLog, N, Sym); report->addRange(ErrorRange); C.emitReport(std::move(report)); } @@ -951,9 +950,9 @@ void RetainCountChecker::checkReturnWithRetEffect(const ReturnStmt *S, ExplodedNode *N = C.addTransition(state, Pred, &ReturnOwnLeakTag); if (N) { const LangOptions &LOpts = C.getASTContext().getLangOpts(); - C.emitReport(std::unique_ptr<BugReport>(new CFRefLeakReport( - *getLeakAtReturnBug(LOpts), LOpts, - SummaryLog, N, Sym, C, IncludeAllocationLine))); + C.emitReport(llvm::make_unique<CFRefLeakReport>( + *getLeakAtReturnBug(LOpts), LOpts, SummaryLog, N, Sym, C, + IncludeAllocationLine)); } } } @@ -978,9 +977,9 @@ void RetainCountChecker::checkReturnWithRetEffect(const ReturnStmt *S, if (!returnNotOwnedForOwned) returnNotOwnedForOwned.reset(new ReturnedNotOwnedForOwned(this)); - C.emitReport(std::unique_ptr<BugReport>(new CFRefReport( + C.emitReport(llvm::make_unique<CFRefReport>( *returnNotOwnedForOwned, C.getASTContext().getLangOpts(), - SummaryLog, N, Sym))); + SummaryLog, N, Sym)); } } } @@ -1182,9 +1181,8 @@ RetainCountChecker::handleAutoreleaseCounts(ProgramStateRef state, overAutorelease.reset(new OverAutorelease(this)); const LangOptions &LOpts = Ctx.getASTContext().getLangOpts(); - Ctx.emitReport(std::unique_ptr<BugReport>( - new CFRefReport(*overAutorelease, LOpts, - SummaryLog, N, Sym, os.str()))); + Ctx.emitReport(llvm::make_unique<CFRefReport>( + *overAutorelease, LOpts, SummaryLog, N, Sym, os.str())); } return nullptr; @@ -1235,9 +1233,8 @@ RetainCountChecker::processLeaks(ProgramStateRef state, : getLeakAtReturnBug(LOpts); assert(BT && "BugType not initialized."); - Ctx.emitReport(std::unique_ptr<BugReport>( - new CFRefLeakReport(*BT, LOpts, SummaryLog, N, *I, Ctx, - IncludeAllocationLine))); + Ctx.emitReport(llvm::make_unique<CFRefLeakReport>( + *BT, LOpts, SummaryLog, N, *I, Ctx, IncludeAllocationLine)); } } |

