summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Jackson <ian.jackson@eu.citrix.com>2018-08-22 13:53:39 +0100
committerIan Jackson <Ian.Jackson@eu.citrix.com>2018-08-22 15:13:29 +0100
commit4a7017c067cbe0e81243c0e4876c8e8248be26ba (patch)
tree6f45ca9f04ed420649082b3e326879c80fb29b27
parent9b2cb3ca974268d7b1100afe0350f29574ba1c67 (diff)
Do not send "\e[21m" when turning off bold. It enables underline!
Specifically, in ECMA-48, ESC [ 21 m means to turn on double underline. The result is that much output is underlined. According to https://bjh21.me.uk/all-escapes/all-escapes.txt this sequence is or was used by `linux' to turn off bold. But sympathy also sends ESC [ 22 m. Tests with Linux 3.16.43-2+deb8u3 show that a Linux console reacts to ESC [ 22 m by turning off bold, and that ESC [ 21 m is not needed. Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
-rw-r--r--src/ansi.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/src/ansi.c b/src/ansi.c
index 91a80b6..1c7fe57 100644
--- a/src/ansi.c
+++ b/src/ansi.c
@@ -370,8 +370,6 @@ ansi_set_attr (ANSI * a, int attr)
if (a->terminal->xmit (a->terminal, "\033[1m", 4) != 4)
err++;
} else {
- if (a->terminal->xmit (a->terminal, "\033[21m", 5) != 5)
- err++;
if (a->terminal->xmit (a->terminal, "\033[22m", 5) != 5)
err++;
}