Use DeclStmt::decl_iterator instead of walking the getNextDeclarator() chain.

llvm-svn: 54501
This commit is contained in:
Ted Kremenek 2008-08-08 02:45:18 +00:00
parent 4b1327960d
commit 4a35180f6d
1 changed files with 4 additions and 4 deletions

View File

@ -526,13 +526,13 @@ Sema::ActOnForStmt(SourceLocation ForLoc, SourceLocation LParenLoc,
if (DeclStmt *DS = dyn_cast_or_null<DeclStmt>(First)) { if (DeclStmt *DS = dyn_cast_or_null<DeclStmt>(First)) {
// C99 6.8.5p3: The declaration part of a 'for' statement shall only declare // C99 6.8.5p3: The declaration part of a 'for' statement shall only declare
// identifiers for objects having storage class 'auto' or 'register'. // identifiers for objects having storage class 'auto' or 'register'.
for (ScopedDecl *D = DS->getDecl(); D; D = D->getNextDeclarator()) { for (DeclStmt::decl_iterator DI=DS->decl_begin(), DE=DS->decl_end();
VarDecl *VD = dyn_cast<VarDecl>(D); DI!=DE; ++DI) {
VarDecl *VD = dyn_cast<VarDecl>(*DI);
if (VD && VD->isBlockVarDecl() && !VD->hasLocalStorage()) if (VD && VD->isBlockVarDecl() && !VD->hasLocalStorage())
VD = 0; VD = 0;
if (VD == 0) if (VD == 0)
Diag(dyn_cast<ScopedDecl>(D)->getLocation(), Diag((*DI)->getLocation(), diag::err_non_variable_decl_in_for);
diag::err_non_variable_decl_in_for);
// FIXME: mark decl erroneous! // FIXME: mark decl erroneous!
} }
} }