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:57 | Date Closed: | 2015-05-17 15:25:47 |
Priority: | Minor | Regression? | |
Status: | Closed/Complete | Components: | 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. |