On Aug 17, 2012, at 5:49 PM, Dmitri Gribenko <[email protected]> wrote:

> Hello,
> 
> The attached patch adds a flag
> PrintingPolicy::DontRecurseInDeclContext to provide "terse" output
> from DeclPrinter.  The motivation is to use DeclPrinter to print
> declarations in user-friendly format, without overwhelming user with
> inner detail of the declaration being printed.
> 
> This patch also adds many tests for DeclPrinter.  There are quite a
> few things that we print incorrectly: search for WRONG in test file --
> and these tests check our output against incorrect output, so that we
> can fix/refactor/rewrite the DeclPrinter later.


Patch LGTM. We can evolve the tests and the DeclPrint to be less broken over 
time. Thanks!


        - Doug
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to