[cfe-commits] r69437 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDecl.cpp test/Sema/scope-check.c test/SemaObjC/scope-check-try-catch.m

[Eli Friedman]

On Sat, Apr 18, 2009 at 2:36 AM, Chris Lattner <sabre at nondot.org> wrote:
> +    // Disallow jumps into any part of an @try statement by pushing a scope and
> +    // walking all sub-stmts in that scope.
> +    if (ObjCAtTryStmt *AT = dyn_cast<ObjCAtTryStmt>(SubStmt)) {
> +      Scopes.push_back(GotoScope(ParentScope, diag::note_protected_by_objc_try,
> +                                 AT->getAtTryLoc()));
> +      // Recursively walk the AST.
> +      BuildScopeInformation(SubStmt, Scopes.size()-1);
> +      continue;
>     }

I think this doesn't catch "@try { goto L; } @catch (...) {L: ;}"; was
that intentional?

Otherwise, looks good.

-Eli