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

[Philip Craig]

It is dumping more than just declarators.

On Mon, Dec 3, 2012 at 8:30 PM, Chandler Carruth <chandlerc at google.com> wrote:
> 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
>>
>