diff options
author | Ted Kremenek <kremenek@apple.com> | 2009-08-07 20:55:20 +0000 |
---|---|---|
committer | Ted Kremenek <kremenek@apple.com> | 2009-08-07 20:55:20 +0000 |
commit | d99f2866ac217c8219f06fb090ad0e82752d939e (patch) | |
tree | 97dbf0f67826d553561a603ba09961f7cafbe064 /clang/lib/Analysis/CheckObjCUnusedIVars.cpp | |
parent | bfdd58e103373ebc0fbe83bfc39b9e2a7543e1c7 (diff) | |
download | bcm5719-llvm-d99f2866ac217c8219f06fb090ad0e82752d939e.tar.gz bcm5719-llvm-d99f2866ac217c8219f06fb090ad0e82752d939e.zip |
Constify some pointers. No functionality change.
llvm-svn: 78407
Diffstat (limited to 'clang/lib/Analysis/CheckObjCUnusedIVars.cpp')
-rw-r--r-- | clang/lib/Analysis/CheckObjCUnusedIVars.cpp | 31 |
1 files changed, 16 insertions, 15 deletions
diff --git a/clang/lib/Analysis/CheckObjCUnusedIVars.cpp b/clang/lib/Analysis/CheckObjCUnusedIVars.cpp index 0063c40482a..dbc90d65c30 100644 --- a/clang/lib/Analysis/CheckObjCUnusedIVars.cpp +++ b/clang/lib/Analysis/CheckObjCUnusedIVars.cpp @@ -24,46 +24,48 @@ using namespace clang; enum IVarState { Unused, Used }; -typedef llvm::DenseMap<ObjCIvarDecl*,IVarState> IvarUsageMap; +typedef llvm::DenseMap<const ObjCIvarDecl*,IVarState> IvarUsageMap; -static void Scan(IvarUsageMap& M, Stmt* S) { +static void Scan(IvarUsageMap& M, const Stmt* S) { if (!S) return; - if (ObjCIvarRefExpr* Ex = dyn_cast<ObjCIvarRefExpr>(S)) { - ObjCIvarDecl* D = Ex->getDecl(); + if (const ObjCIvarRefExpr* Ex = dyn_cast<ObjCIvarRefExpr>(S)) { + const ObjCIvarDecl* D = Ex->getDecl(); IvarUsageMap::iterator I = M.find(D); - if (I != M.end()) I->second = Used; + if (I != M.end()) + I->second = Used; return; } - for (Stmt::child_iterator I=S->child_begin(), E=S->child_end(); I!=E;++I) + for (Stmt::const_child_iterator I=S->child_begin(),E=S->child_end(); I!=E;++I) Scan(M, *I); } -static void Scan(IvarUsageMap& M, ObjCPropertyImplDecl* D) { +static void Scan(IvarUsageMap& M, const ObjCPropertyImplDecl* D) { if (!D) return; - ObjCIvarDecl* ID = D->getPropertyIvarDecl(); + const ObjCIvarDecl* ID = D->getPropertyIvarDecl(); if (!ID) return; IvarUsageMap::iterator I = M.find(ID); - if (I != M.end()) I->second = Used; + if (I != M.end()) + I->second = Used; } void clang::CheckObjCUnusedIvar(ObjCImplementationDecl* D, BugReporter& BR) { - ObjCInterfaceDecl* ID = D->getClassInterface(); + const ObjCInterfaceDecl* ID = D->getClassInterface(); IvarUsageMap M; // Iterate over the ivars. - for (ObjCInterfaceDecl::ivar_iterator I=ID->ivar_begin(), E=ID->ivar_end(); - I!=E; ++I) { + for (ObjCInterfaceDecl::ivar_iterator I=ID->ivar_begin(), + E=ID->ivar_end(); I!=E; ++I) { - ObjCIvarDecl* ID = *I; + const ObjCIvarDecl* ID = *I; // Ignore ivars that aren't private. if (ID->getAccessControl() != ObjCIvarDecl::Private) @@ -81,7 +83,7 @@ void clang::CheckObjCUnusedIvar(ObjCImplementationDecl* D, BugReporter& BR) { // Now scan the methods for accesses. for (ObjCImplementationDecl::instmeth_iterator I = D->instmeth_begin(), - E = D->instmeth_end(); I!=E; ++I) + E = D->instmeth_end(); I!=E; ++I) Scan(M, (*I)->getBody()); // Scan for @synthesized property methods that act as setters/getters @@ -104,4 +106,3 @@ void clang::CheckObjCUnusedIvar(ObjCImplementationDecl* D, BugReporter& BR) { os.str().c_str(), I->first->getLocation()); } } - |