1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

util: use blue on non-windows systems for number/bigint

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/18925
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
This commit is contained in:
Gus Caplan 2018-02-21 20:45:57 -06:00
parent da886d9a4c
commit 1708af369b
No known key found for this signature in database
GPG Key ID: F00BD11880E82F0E
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/util.js
View File

@ -339,10 +339,11 @@ inspect.colors = Object.assign(Object.create(null), {
});
// Don't use 'blue' not visible on cmd.exe
const windows = process.platform === 'win32';
inspect.styles = Object.assign(Object.create(null), {
'special': 'cyan',
'number': 'yellow',
'bigint': 'yellow',
'number': windows ? 'yellow' : 'blue',
'bigint': windows ? 'yellow' : 'blue',
'boolean': 'yellow',
'undefined': 'grey',
'null': 'bold',

3
test/parallel/test-stream-buffer-list.js
View File

@ -34,7 +34,8 @@ assert.deepStrictEqual(list, new BufferList());
const tmp = util.inspect.defaultOptions.colors;
util.inspect.defaultOptions = { colors: true };
const color = util.inspect.colors[util.inspect.styles.number];
assert.strictEqual(
util.inspect(list),
'BufferList { length: \u001b[33m0\u001b[39m }');
`BufferList { length: \u001b[${color[0]}m0\u001b[${color[1]}m }`);
util.inspect.defaultOptions = { colors: tmp };