aboutsummaryrefslogtreecommitdiff
path: root/src/xz/util.h
diff options
context:
space:
mode:
authorJia Tan <jiat0218@gmail.com>2023-11-23 22:04:35 +0800
committerJia Tan <jiat0218@gmail.com>2023-11-23 22:40:20 +0800
commit584e3a258f32d579b1d07f99b4dc6e856c10ac7e (patch)
treea891ce6e03d4579de0fd0d00714abfd428bdd0c2 /src/xz/util.h
parenttuklib_integer: Fix typo discovered by codespell. (diff)
downloadxz-584e3a258f32d579b1d07f99b4dc6e856c10ac7e.tar.xz
xz: Create separate is_tty() function.
The new is_tty() will report if a file descriptor is a terminal or not. On POSIX systems, it is a wrapper around isatty(). However, the native Windows implementation of isatty() will return true for all character devices, not just terminals. So is_tty() has a special case for Windows so it can use alternative Windows API functions to determine if a file descriptor is a terminal. This fixes a bug with MSVC and MinGW-w64 builds that refused to read from or write to non-terminal character devices because xz thought it was a terminal. For instance: xz foo -c > /dev/null would fail because /dev/null was assumed to be a terminal.
Diffstat (limited to 'src/xz/util.h')
-rw-r--r--src/xz/util.h14
1 files changed, 14 insertions, 0 deletions
diff --git a/src/xz/util.h b/src/xz/util.h
index 6d7e1481..1da40371 100644
--- a/src/xz/util.h
+++ b/src/xz/util.h
@@ -105,6 +105,20 @@ lzma_attribute((__format__(__printf__, 3, 4)))
extern void my_snprintf(char **pos, size_t *left, const char *fmt, ...);
+/// \brief Test if file descriptor is a terminal
+///
+/// For POSIX systems, this is a simple wrapper around isatty(). However on
+/// Windows, isatty() returns true for all character devices, not just
+/// terminals.
+///
+/// \param fd File descriptor to test
+///
+/// \return bool:
+/// - true if file descriptor is a terminal
+/// - false otherwise
+extern bool is_tty(int fd);
+
+
/// \brief Test if stdin is a terminal
///
/// If stdin is a terminal, an error message is printed and exit status set