On Mon, Aug 20, 2012 at 4:33 PM, Douglas Gregor <[email protected]> wrote:
> On Aug 17, 2012, at 5:49 PM, Dmitri Gribenko <[email protected]> wrote:
>> 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!
Thank you for the review, committed r162245.
Dmitri
--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
