Thanks, committed in r128371 with a couple of style tweaks.

On Wed, Mar 16, 2011 at 6:58 PM, Richard Trieu <[email protected]> wrote:

> This patch adds flags to print or omit the include stacks of notes.  These
> stacks are less helpful in notes than those in errors and warnings.  Below
> is the current output on an error, and how it will show up when the include
> stack is disabled.
>
> Current behavior:
>
> test/test.cpp:9:10: error: no matching function for
>       call to 'foo'
>   return foo(1, 1);
>          ^~~
> In file included from test/test.cpp:7:
> In file included from test/include7.h:1:
> In file included from test/include6.h:1:
> In file included from test/include5.h:1:
> In file included from test/include4.h:1:
> In file included from test/include3.h:1:
> In file included from test/include2.h:1:
> include1.h:1:5: note: candidate function not viable: requires 1
>       argument, but 2 were provided
> int foo(int x) { return x; }
>
>
> With new flag to omit note include stacks:
>
> test/test.cpp:9:10: error: no matching function for
>       call to 'foo'
>   return foo(1, 1);
>          ^~~
> include1.h:1:5: note: candidate function not viable: requires 1
>       argument, but 2 were provided
> int foo(int x) { return x; }
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to