diff: Only color output when stdout is a tty

If we're redirecting the output somewhere, we likely don't want to
have ANSI codes in the middle of our diff output.

1 files changed, 14 insertions, 4 deletions

diff --git a/Userland/Utilities/diff.cpp b/Userland/Utilities/diff.cpp
index 2ea0d7b8ee..cfbaf02d49 100644
--- a/Userland/Utilities/diff.cpp
+++ b/Userland/Utilities/diff.cpp
@@ -61,13 +61,23 @@ int main(int argc, char** argv)
         if (num_added > 1)
             sb.appendff(",{}", target_start + num_added - 1);
 
+        bool color_output = isatty(STDOUT_FILENO);
+
         outln("Hunk: {}", sb.build());
-        for (const auto& line : hunk.removed_lines)
-            outln("\033[31;1m< {}\033[0m", line);
+        for (const auto& line : hunk.removed_lines) {
+            if (color_output)
+                outln("\033[31;1m< {}\033[0m", line);
+            else
+                outln("< {}", line);
+        }
         if (num_added > 0 && num_removed > 0)
             outln("---");
-        for (const auto& line : hunk.added_lines)
-            outln("\033[32;1m> {}\033[0m", line);
+        for (const auto& line : hunk.added_lines) {
+            if (color_output)
+                outln("\033[32;1m> {}\033[0m", line);
+            else
+                outln("> {}", line);
+        }
     }
 
     return hunks.is_empty() ? 0 : 1;