From: Ted Kremenek Date: Fri, 7 Aug 2009 20:55:20 +0000 (+0000) Subject: Constify some pointers. No functionality change. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0b2dd776318e612c682d32e1f4ca70f7b223c05e;p=clang Constify some pointers. No functionality change. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@78407 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Analysis/CheckObjCUnusedIVars.cpp b/lib/Analysis/CheckObjCUnusedIVars.cpp index 0063c40482..dbc90d65c3 100644 --- a/lib/Analysis/CheckObjCUnusedIVars.cpp +++ b/lib/Analysis/CheckObjCUnusedIVars.cpp @@ -24,46 +24,48 @@ using namespace clang; enum IVarState { Unused, Used }; -typedef llvm::DenseMap IvarUsageMap; +typedef llvm::DenseMap IvarUsageMap; -static void Scan(IvarUsageMap& M, Stmt* S) { +static void Scan(IvarUsageMap& M, const Stmt* S) { if (!S) return; - if (ObjCIvarRefExpr* Ex = dyn_cast(S)) { - ObjCIvarDecl* D = Ex->getDecl(); + if (const ObjCIvarRefExpr* Ex = dyn_cast(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()); } } -