Skip to content

Commit 1a7cbf1

Browse files
committed
Fix formatting.
1 parent ffeb0f6 commit 1a7cbf1

File tree

1 file changed

+14
-13
lines changed

1 file changed

+14
-13
lines changed

clang/lib/StaticAnalyzer/Checkers/WebKit/RetainPtrCtorAdoptChecker.cpp

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -125,11 +125,11 @@ class RetainPtrCtorAdoptChecker
125125
return isAdoptFnName(safeGetName(FnDecl));
126126
}
127127

128-
bool isAdoptFnName(const std::string& Name) const {
128+
bool isAdoptFnName(const std::string &Name) const {
129129
return isAdoptNS(Name) || Name == "adoptCF" || Name == "adoptCFArc";
130130
}
131131

132-
bool isAdoptNS(const std::string& Name) const {
132+
bool isAdoptNS(const std::string &Name) const {
133133
return Name == "adoptNS" || Name == "adoptNSArc";
134134
}
135135

@@ -165,7 +165,7 @@ class RetainPtrCtorAdoptChecker
165165
checkCreateOrCopyFunction(CE, DeclWithIssue);
166166
}
167167

168-
void checkAdoptCall(const CallExpr *CE, const std::string& FnName,
168+
void checkAdoptCall(const CallExpr *CE, const std::string &FnName,
169169
const Decl *DeclWithIssue) const {
170170
if (!CE->getNumArgs())
171171
return;
@@ -259,20 +259,21 @@ class RetainPtrCtorAdoptChecker
259259
if (ParamDecl->hasAttr<CFReturnsRetainedAttr>())
260260
CreateOrCopyOutArguments.insert(Decl);
261261
} else {
262-
// No callee or a variadic argument. Conservatively assume it's an out argument.
262+
// No callee or a variadic argument.
263+
// Conservatively assume it's an out argument.
263264
if (RTC.isUnretained(Decl->getType()))
264265
CreateOrCopyOutArguments.insert(Decl);
265266
}
266267
}
267268
auto Summary = Summaries->getSummary(AnyCall(CE));
268269
switch (Summary->getRetEffect().getKind()) {
269-
case RetEffect::OwnedSymbol:
270-
case RetEffect::OwnedWhenTrackedReceiver:
271-
if (!CreateOrCopyFnCall.contains(CE))
272-
reportLeak(CE, DeclWithIssue);
273-
break;
274-
default:
275-
break;
270+
case RetEffect::OwnedSymbol:
271+
case RetEffect::OwnedWhenTrackedReceiver:
272+
if (!CreateOrCopyFnCall.contains(CE))
273+
reportLeak(CE, DeclWithIssue);
274+
break;
275+
default:
276+
break;
276277
}
277278
}
278279

@@ -340,7 +341,7 @@ class RetainPtrCtorAdoptChecker
340341
else if (isCreateOrCopy(Arg))
341342
reportLeak(Name, CE, DeclWithIssue);
342343
}
343-
344+
344345
void visitVarDecl(const VarDecl *VD) const {
345346
auto *Init = VD->getInit();
346347
if (!Init || !RTC.isARCEnabled())
@@ -446,7 +447,7 @@ class RetainPtrCtorAdoptChecker
446447
auto *Receiver = ObjCMsgExpr->getInstanceReceiver()->IgnoreParenCasts();
447448
if (!Receiver)
448449
return false;
449-
if (auto *Inner = dyn_cast<ObjCMessageExpr>(Receiver)) {
450+
if (auto *Inner = dyn_cast<ObjCMessageExpr>(Receiver)) {
450451
if (InnerExpr)
451452
*InnerExpr = Inner;
452453
auto InnerSelector = Inner->getSelector();

0 commit comments

Comments
 (0)