[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