From 48b9fdc152f655c7b35b1b49155ef0990ff8de9a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rainer=20M=C3=BCller?= Date: Tue, 5 Mar 2013 21:02:00 -0500 Subject: [PATCH] [PATCH] Bug MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Do not print xterm titles without a tty. Writing escape sequences for xterm titles is unnecessary when the output is piped into another tool. Signed-off-by: Paul Beckingham --- ChangeLog | 2 ++ src/Context.cpp | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index bade9ddd6..57964e6bf 100644 --- a/ChangeLog +++ b/ChangeLog @@ -113,6 +113,8 @@ Bugs + Added missing localized confirmation strings (thanks to Rainer Müller). + Fixed bug that added an extra newline after setting the terminal title (thanks to Rainer Müller). + + Fixed a bug where the terminal title was tried to be set without a tty + (thanks to Rainer Müller). ------ old releases ------------------------------ diff --git a/src/Context.cpp b/src/Context.cpp index 014a26ba6..60c72122e 100644 --- a/src/Context.cpp +++ b/src/Context.cpp @@ -676,7 +676,7 @@ void Context::clear () // this output?'. void Context::updateXtermTitle () { - if (config.getBoolean ("xterm.title")) + if (config.getBoolean ("xterm.title") && isatty (fileno (stdout))) { std::string command; a3.find_command (command);