<div class="gmail_quote">On Sun, Mar 27, 2011 at 10:37 AM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
On Mar 26, 2011, at 6:50 PM, Chandler Carruth wrote:<br>
<br>
> Author: chandlerc<br>
> Date: Sat Mar 26 20:50:55 2011<br>
> New Revision: 128371<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=128371&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=128371&view=rev</a><br>
> Log:<br>
> Add an option to suppress include stack printing on note diagnostics.<br>
> These stacks are often less important than those on primary diagnostics.<br>
><br>
> As the number of notes grows, this becomes increasingly important. The<br>
> include stack printing is clever and doesn't print stacks for adjacent<br>
> diagnostics from the same file, but when a note is in between a sequence<br>
> of errors in a header file, and the notes all refer to some other file,<br>
> we end up getting a worst-case ping-pong of include stacks that take up<br>
> a great deal of vertical space.<br>
><br>
> Still, for now, the default behavior isn't changed. We can evaluate user<br>
> feedback with the flag.<br>
><br>
> Patch by Richard Trieu, a couple of style tweaks from me.<br>
<br>
</div>When is printing the include stack for a note ever a good idea?  I think we should just remove it unconditionally.<br></blockquote><div><br></div><div>That'll teach me to assume that these things were intentional... ;] I'll flip the default, and if no one complains, remove it. Sound good?</div>
</div>