urxvt supports colors

10 years ago · 1197daf357
parent f8c44a0ae4
commit 1197daf357
2 changed files with 8 additions and 0 deletions
--- a/src/gtest.cc
+++ b/src/gtest.cc
@ -2886,6 +2886,8 @@ bool ShouldUseColor(bool stdout_is_tty) {
        String::CStringEquals(term, "xterm-256color") ||
        String::CStringEquals(term, "screen") ||
        String::CStringEquals(term, "screen-256color") ||
+        String::CStringEquals(term, "rxvt-unicode") ||
+        String::CStringEquals(term, "rxvt-unicode-256color") ||
        String::CStringEquals(term, "linux") ||
        String::CStringEquals(term, "cygwin");
    return stdout_is_tty && term_supports_color;
--- a/test/gtest_unittest.cc
+++ b/test/gtest_unittest.cc
@ -6672,6 +6672,12 @@ TEST(ColoredOutputTest, UsesColorsWhenTermSupportsColors) {
  SetEnv("TERM", "screen-256color");  // TERM supports colors.
  EXPECT_TRUE(ShouldUseColor(true));  // Stdout is a TTY.

+  SetEnv("TERM", "rxvt-unicode");  // TERM supports colors.
+  EXPECT_TRUE(ShouldUseColor(true));  // Stdout is a TTY.
+
+  SetEnv("TERM", "rxvt-unicode-256color");  // TERM supports colors.
+  EXPECT_TRUE(ShouldUseColor(true));  // Stdout is a TTY.
+
  SetEnv("TERM", "linux");  // TERM supports colors.
  EXPECT_TRUE(ShouldUseColor(true));  // Stdout is a TTY.