<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 23, 2013 at 8:16 AM, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">__sanitizer_report_error_summary is the replaceable interface for reporting an error. This patch changes asan a tiny bit to always emit a summary whenever it encounters an error. This is important, otherwise a library user of asan will not be aware of errors that were merely printed.<div>


<br></div><div>Also, all these summaries are redundant with the non-summarized printouts. Make the default weak implementation of report error summary not do anything. This means we need to remove the check for SUMMARY lines from the tests. This is suboptimal for testing, but printing out these redundant lines is just bad UI.</div>
</div></blockquote><div><br></div><div>But I've added summaries for a reason: they help to cluster bug reports.</div><div>I'd really prefer to keep them by default. </div><div><br></div><div>--kcc </div><div><br></div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">

<div><br></div><div>Please review!</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Nick</div><div><br></div></font></span></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>