From 5439f36c976ac31f7faa27cc1536a0ecd3c4469f Mon Sep 17 00:00:00 2001 From: Ryan Liptak Date: Sat, 17 Jun 2023 23:36:10 -0700 Subject: [PATCH] Remove unnecessary isatty call from supportsAnsiEscapeCodes isatty on Windows is implemented as a isCygwinPty call and a GetConsoleMode call, so calling isatty just duplicates the function calls we already need to do in supportsAnsiEscapeCodes. --- lib/std/fs/file.zig | 2 -- 1 file changed, 2 deletions(-) diff --git a/lib/std/fs/file.zig b/lib/std/fs/file.zig index 0a2fbf74ad..7b954399d8 100644 --- a/lib/std/fs/file.zig +++ b/lib/std/fs/file.zig @@ -230,8 +230,6 @@ pub const File = struct { /// Test whether ANSI escape codes will be treated as such. pub fn supportsAnsiEscapeCodes(self: File) bool { if (builtin.os.tag == .windows) { - if (!os.isatty(self.handle)) return false; - var console_mode: os.windows.DWORD = 0; if (os.windows.kernel32.GetConsoleMode(self.handle, &console_mode) != 0) { if (console_mode & os.windows.ENABLE_VIRTUAL_TERMINAL_PROCESSING != 0) return true;