On Wed, Jan 23, 2013 at 12:12 PM, Philip Craig <[email protected]>wrote:
>
>
> ================
> Comment at: include/clang/AST/DeclBase.h:853
> @@ -852,2 +852,3 @@
> // Debuggers don't usually respect default arguments.
> LLVM_ATTRIBUTE_USED void dump() const;
> + // Same as dump(), but forces color printing.
> ----------------
> Saleem Abdulrasool wrote:
> > Why not make colour a default valued parameter for dump? This would
> avoid duplication of the method as well as reduce the interface.
> As the comment states, debuggers don't handle default arguments.
>
> ================
> Comment at: lib/AST/ASTDumper.cpp:113
> @@ -58,1 +112,3 @@
> + LastLocFilename(""), LastLocLine(~0U),
> + ShowColors(SM && SM->getDiagnostics().getShowColors()) { }
>
> ----------------
> I used OS.has_colors() in lib/Frontend/ASTConsumers.cpp. We probably
> should be consistent between these. I don't know which is better.
>
>
> http://llvm-reviews.chandlerc.com/D291
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
Is OS.has_colors() correctly set in case of pipes ? If not then it makes
sense to use a user available flag.
-- Matthieu
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
