Let lsan honor color option

[Sergey Matveev]

Thanks, fixed in r210199.

Generally, compiler-rt patches should be posted to llvm-commits, not
cfe-commits.


On Wed, Jun 4, 2014 at 8:12 PM, Stephan Bergmann <sbergman at redhat.com>
wrote:

> The below change to trunk projects/compiler-rt makes
> ASAN_OPTIONS=detect_leaks=1:color=always colorize the LeakSanitizer
> output as intended:
>
>  Index: lib/lsan/lsan_common.cc
>> ===================================================================
>> --- lib/lsan/lsan_common.cc     (revision 210183)
>> +++ lib/lsan/lsan_common.cc     (working copy)
>> @@ -133,7 +133,7 @@
>>
>>  class Decorator: private __sanitizer::AnsiColorDecorator {
>>   public:
>> -  Decorator() : __sanitizer::AnsiColorDecorator(PrintsToTtyCached()) { }
>> +  Decorator() : __sanitizer::AnsiColorDecorator(ColorizeReports()) { }
>>    const char *Error() { return Red(); }
>>    const char *Leak() { return Blue(); }
>>    const char *End() { return Default(); }
>>
>
> Stephan
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140604/d1073927/attachment.html>