diff options
| author | D Tucny <d@tucny.com> | 2015-05-01 20:50:22 -0400 |
|---|---|---|
| committer | Matt Jordan <mjordan@digium.com> | 2015-05-03 09:41:15 -0500 |
| commit | 92120247e99865d97a85461b08ba763e961e6232 (patch) | |
| tree | bd5b7d4a3a6f683a1748a1f224706379a1193f01 /main/term.c | |
| parent | 13819a34c4d0cd151da60502ef008dcde55b6dca (diff) | |
term: send proper reset sequence when black background is forced
When using the force black background command-line option or configuration
option an invalid reset sequence is sent following a coloured output item
in the CLI, the result is that the colour is not 'turned off' and continues
until the next non-default coloured text output.
A reset sequence is already defined in term.c, but the ast_term_reset
function doesn't use it, instead building it's own invalid sequence and
returning that.
This patch changes that behaviour, removing the building of a reset sequence
and instead using the pre-built constant 'enddata' which is a suitable reset
sequence for this purpose.
ASTERISK-24896 #close
Reported by: Dan Tucny
Change-Id: I56323899123ae3264900389cae1f5b252aa3bf43
Diffstat (limited to 'main/term.c')
| -rw-r--r-- | main/term.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/main/term.c b/main/term.c index 2e28032a7..871a7cb3c 100644 --- a/main/term.c +++ b/main/term.c @@ -311,8 +311,7 @@ const char *ast_term_color(int fgcolor, int bgcolor) const char *ast_term_reset(void) { if (ast_opt_force_black_background) { - static const char reset[] = { ESC, '[', COLOR_BLACK + 10, 'm', 0 }; - return reset; + return enddata; } else { return quitdata; } |