[Lldb-commits] [PATCH] D51615: Set Windows console mode to enable support for ansi escape codes

Stella Stamenova via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Sep 6 14:14:59 PDT 2018

stella.stamenova added a comment.

In https://reviews.llvm.org/D51615#1226396, @xbolva00 wrote:

> Bot sure how to fix tests since this is somehow hard to problematic.

I think the fix for this is simply to not make the call to SetUseColor here. Is there a reason for the call? It looks like the check for "dumb" terminal and "GetIsTerminalWithColors" might have to come after setting the ANSI escape codes if setting the codes impacts those two cases.



More information about the lldb-commits mailing list