TheThing/node-bunyan-lite
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Wiki Activity

[issue #59] Clear a possibly interrupted ANSI color code on signal termination.

Browse source
This commit is contained in:
Trent Mick 2012-11-14 19:27:53 -08:00
parent af002db218
commit 70f8d4d4da
2 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
CHANGES.md
View file

@ -2,7 +2,8 @@
## bunyan 0.16.7 (not yet released) ## bunyan 0.16.7 (not yet released)
(nothing yet) - [issue #59] Clear a possibly interrupted ANSI color code on signal
termination.
## bunyan 0.16.6 ## bunyan 0.16.6

8
bin/bunyan
View file

@ -73,6 +73,9 @@ var currLine = null;
// Child dtrace process, if any. Used for signal-handling. // Child dtrace process, if any. Used for signal-handling.
var child = null; var child = null;
// Whether ANSI codes are being used. Used for signal-handling.
var usingAnsiCodes = false;
//---- support functions //---- support functions
@ -1147,6 +1150,10 @@ function asyncForEach(arr, iterator, callback) {
// Try to ensure we close a child 'dtrace' process on signalled exit. // Try to ensure we close a child 'dtrace' process on signalled exit.
function signalCleanupAndExit(signal) { function signalCleanupAndExit(signal) {
if (usingAnsiCodes) {
// Clear possibly interrupted ANSI code (issue #59).
process.stdout.write('\033[0m');
}
if (child) { if (child) {
child.kill(signal); child.kill(signal);
} }
@ -1215,6 +1222,7 @@ function main(argv) {
opts.color = process.stdout.isTTY; opts.color = process.stdout.isTTY;
} }
} }
usingAnsiCodes = opts.color; // intentionally global
var stylize = (opts.color ? stylizeWithColor : stylizeWithoutColor); var stylize = (opts.color ? stylizeWithColor : stylizeWithoutColor);
var retval = 0; var retval = 0;