[Home]

Summary:ASTERISK-24896: [patch] Using force black background leads to colours not being reset
Reporter:dant (dant)Labels:
Date Opened:2015-03-19 19:39:57Date Closed:2015-05-17 15:25:47
Priority:MinorRegression?
Status:Closed/CompleteComponents:Core/General
Versions:13.2.0 Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) ASTERISK-24896-term.c.patch
Description:When using force black background the reset sequence used only resets the background leaving the foreground to bleed through until something else changes the foreground.
Comments:By: dant (dant) 2015-03-19 19:44:00.434-0500

Attached patch uses the already defined 'enddata' string to reset colours which does a complete reset, sets foreground to default and sets background to default.

By: Rusty Newton (rnewton) 2015-03-24 18:05:15.317-0500

Thanks for the contribution! If you'd like your contribution to be included faster, you should submit your patch for code review by the Asterisk Developer Community. To do so, please follow the Code Review [1] instructions on the wiki. Be sure to:
* Verify that your patch conforms to the Coding Guidelines [2]
* Review the Code Review Checklist [3] for common items reviewers will look for
* If necessary, provide tests for the Asterisk Test Suite that verify the correctness of your patch [4]

When ready, submit your patch and any tests to Review Board [5] for code review.

Thanks!

[1] https://wiki.asterisk.org/wiki/display/AST/Code+Review
[2] https://wiki.asterisk.org/wiki/display/AST/Coding+Guidelines
[3] https://wiki.asterisk.org/wiki/display/AST/Code+Review+Checklist
[4] https://wiki.asterisk.org/wiki/display/AST/Asterisk+Test+Suite+Documentation
[5] https://wiki.asterisk.org/wiki/display/AST/Review+Board+Usage



By: Rusty Newton (rnewton) 2015-05-17 15:25:47.113-0500

Auto-close wasn't working. Closing this out since the fix was merged.