[cfe-commits] r69895 - /cfe/trunk/lib/Analysis/BugReporter.cpp

Ted Kremenek kremenek at apple.com
Fri Apr 24 08:31:25 PDT 2009


On Apr 23, 2009, at 9:43 AM, Daniel Dunbar wrote:

> On Thu, Apr 23, 2009 at 9:19 AM, Ted Kremenek <kremenek at apple.com>  
> wrote:
> +        // Ignore parentheses.
> +        if (const ParenExpr *PE = dyn_cast<ParenExpr>(S))
> +          S = PE->IgnoreParens();
>
> The dyn_cast here is unneeded, IgnoreParens is defined on Expr?
>
>  - Daniel
>

Yes, IgnoreParens is defined in Expr.  'S' is a Stmt*.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20090424/d35c3353/attachment.html>


More information about the cfe-commits mailing list