[cfe-commits] [PATCH][Review Request] Handle CXXForRangeStmt

Ted Kremenek kremenek at apple.com
Tue Oct 4 15:20:49 PDT 2011


Test case please.  :)

On Oct 4, 2011, at 1:32 PM, Jim Goodnow II wrote:

> This is just a program control statement and should be handled the same as any other.
> 
> - jim
> 
> Index: lib/StaticAnalyzer/Core/ExprEngine.cpp
> ===================================================================
> --- lib/StaticAnalyzer/Core/ExprEngine.cpp    (revision 141095)
> +++ lib/StaticAnalyzer/Core/ExprEngine.cpp    (working copy)
> @@ -453,7 +453,6 @@
>     case Stmt::CXXBindTemporaryExprClass:
>     case Stmt::CXXCatchStmtClass:
>     case Stmt::CXXDependentScopeMemberExprClass:
> -    case Stmt::CXXForRangeStmtClass:
>     case Stmt::CXXPseudoDestructorExprClass:
>     case Stmt::CXXTemporaryObjectExprClass:
>     case Stmt::CXXThrowExprClass:
> @@ -501,6 +500,7 @@
>     case Stmt::CaseStmtClass:
>     case Stmt::CompoundStmtClass:
>     case Stmt::ContinueStmtClass:
> +    case Stmt::CXXForRangeStmtClass:
>     case Stmt::DefaultStmtClass:
>     case Stmt::DoStmtClass:
>     case Stmt::ForStmtClass:
> 
> <ForRange.patch>




More information about the cfe-commits mailing list