[PATCH] D81110: [Support] Move color handling from raw_fd_ostream to raw_ostream

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 4 12:09:42 PDT 2020


JDevlieghere updated this revision to Diff 268552.
JDevlieghere added a comment.

Implement a variation of Pavel's suggestion. I've added a method called `use_color()` which computer whether we should use colors are not. I hope the inline comments are sufficiently self-explanatory.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81110/new/

https://reviews.llvm.org/D81110

Files:
  llvm/include/llvm/Support/raw_ostream.h
  llvm/lib/Support/raw_ostream.cpp
  llvm/unittests/Support/raw_ostream_test.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81110.268552.patch
Type: text/x-patch
Size: 7219 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200604/7048aa52/attachment-0001.bin>


More information about the llvm-commits mailing list