<br><br><div class="gmail_quote">On Wed, Jan 23, 2013 at 12:12 PM, Philip Craig <span dir="ltr"><<a href="mailto:philipjcraig@gmail.com" target="_blank">philipjcraig@gmail.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>
<br>
================<br>
Comment at: include/clang/AST/DeclBase.h:853<br>
@@ -852,2 +852,3 @@<br>
   // Debuggers don't usually respect default arguments.<br>
   LLVM_ATTRIBUTE_USED void dump() const;<br>
+  // Same as dump(), but forces color printing.<br>
----------------<br>
</div><div class="im">Saleem Abdulrasool wrote:<br>
> Why not make colour a default valued parameter for dump?  This would avoid duplication of the method as well as reduce the interface.<br>
</div>As the comment states, debuggers don't handle default arguments.<br>
<br>
================<br>
Comment at: lib/AST/ASTDumper.cpp:113<br>
@@ -58,1 +112,3 @@<br>
+        LastLocFilename(""), LastLocLine(~0U),<br>
+        ShowColors(SM && SM->getDiagnostics().getShowColors()) { }<br>
<br>
----------------<br>
I used OS.has_colors() in lib/Frontend/ASTConsumers.cpp. We probably should be consistent between these. I don't know which is better.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D291" target="_blank">http://llvm-reviews.chandlerc.com/D291</a><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>
</div></div></blockquote></div><br>Is OS.has_colors() correctly set in case of pipes ? If not then it makes sense to use a user available flag.<br><br>-- Matthieu<br>