zturner added a comment. In https://reviews.llvm.org/D54072#1286811, @zturner wrote:
> So I tried to go down this route, but alas, I still can't come up with a good > way to make color output work, because the Stream is not a `StreamFile` > object, it is a `StreamString` object, even when the output is going to a > terminal. We could have an argument such as `--color` but perhaps it should > be the default and the option should be `--no-color`. I tried some more, and yea this is really not easy. The only `raw_ostream` in LLVM that supports color output is the `raw_fd_ostream`, so we can't even easily set up the forwarder to support any of this. Even if we configure the diagnostics engine to use color output and pass it the forwarder `raw_ostream` it still won't work, because it will just call `raw_ostream::changeColor` which does nothing, since that method is only implemented on `raw_fd_ostream`. I'm open to suggestions, but it would be a real shame to not be able to take advantage of the built-in colorization here. Perhaps we can make this a hidden or unsupported command. https://reviews.llvm.org/D54072 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
