[Lldb-commits] [PATCH] D51772: Check if a terminal supports colors on Windows properly

[Dávid Bolvanský via Phabricator via lldb-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL341746: Check if a terminal supports colors on Windows properly (authored by xbolva00, committed by ).
Herald added a subscriber: llvm-commits.

Changed prior to commit:
  https://reviews.llvm.org/D51772?vs=164359&id=164567#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D51772

Files:
  lldb/trunk/source/Core/Debugger.cpp
  lldb/trunk/source/Host/common/File.cpp


Index: lldb/trunk/source/Host/common/File.cpp
===================================================================
--- lldb/trunk/source/Host/common/File.cpp
+++ lldb/trunk/source/Host/common/File.cpp
@@ -806,6 +806,9 @@
     if (_isatty(fd)) {
       m_is_interactive = eLazyBoolYes;
       m_is_real_terminal = eLazyBoolYes;
+#if defined(ENABLE_VIRTUAL_TERMINAL_PROCESSING)
+      m_supports_colors = eLazyBoolYes;
+#endif
     }
 #else
     if (isatty(fd)) {
Index: lldb/trunk/source/Core/Debugger.cpp
===================================================================
--- lldb/trunk/source/Core/Debugger.cpp
+++ lldb/trunk/source/Core/Debugger.cpp
@@ -812,7 +812,6 @@
   // Enabling use of ANSI color codes because LLDB is using them to highlight
   // text.
   llvm::sys::Process::UseANSIEscapeCodes(true);
-  SetUseColor(true);
 #endif
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51772.164567.patch
Type: text/x-patch
Size: 847 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20180908/017b9600/attachment.bin>