[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
Wed Jun 3 10:58:13 PDT 2020


JDevlieghere created this revision.
JDevlieghere added reviewers: MaskRay, sunfish, jhenderson, labath, amccarth.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.
JDevlieghere added a child revision: D81056: [Support] Replace 'DisableColors' boolean with 'ColorMode' enum.

This moves the color handling code from `raw_fd_ostream` to `raw_ostream`. This was requested in D81058 <https://reviews.llvm.org/D81058>  and will make it possible to unit test `WithColor` as requested in D81056 <https://reviews.llvm.org/D81056>.

With the exception that colors are now counted towards the output characters, this change is NFC.


Repository:
  rLLDB LLDB

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.268251.patch
Type: text/x-patch
Size: 6530 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200603/b0eba02a/attachment.bin>


More information about the llvm-commits mailing list