[Asterisk-code-review] term: send proper reset sequence when black background is fo... (asterisk[master])
dant
asteriskteam at digium.com
Fri May 1 22:17:17 CDT 2015
Hello George Joseph,
I'd like you to reexamine a change. Please visit
https://gerrit.asterisk.org/332
to look at the new patch set (#3).
Change subject: term: send proper reset sequence when black background is forced
......................................................................
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
---
M main/term.c
1 file changed, 1 insertion(+), 2 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/32/332/3
--
To view, visit https://gerrit.asterisk.org/332
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I56323899123ae3264900389cae1f5b252aa3bf43
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: dant <d at tucny.com>
Gerrit-Reviewer: George Joseph <george.joseph at fairview5.com>
More information about the asterisk-code-review
mailing list