[cfe-commits] [PATCH] Rename StmtDumper::DumpSubTree() to dumpStmt()

Chandler Carruth chandlerc at google.com
Mon Dec 3 02:30:09 PST 2012


Just FYI, 'Decl' is an abbreviation of Declaration, not Declarator. Was the
function actually dumping the entire declaration, or was it in fact just
dumping the declarator?


On Mon, Dec 3, 2012 at 2:01 AM, Philip Craig <philipjcraig at gmail.com> wrote:

> Also rename DumpDeclarator() to dumpDecl(). Once Decl dumping is added,
> these will be the two main methods of the class, so this is just for
> consistency in naming.
>
> There was a DumpStmt() method already, but there was no point in having
> it, so I have merged it into VisitStmt(). Similarly, DumpExpr() is merged
> into VisitExpr().
>
> http://llvm-reviews.chandlerc.com/D156
>
> Files:
>   lib/AST/StmtDumper.cpp
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121203/b2752a42/attachment.html>


More information about the cfe-commits mailing list