diff --git a/lib/console.js b/lib/console.js index fee3f8ac0b155d..7d6f9c97299e64 100644 --- a/lib/console.js +++ b/lib/console.js @@ -356,17 +356,27 @@ Console.prototype.table = function(tabularData, properties) { const getIndexArray = (length) => ArrayFrom({ length }, (_, i) => inspect(i)); const mapIter = isMapIterator(tabularData); + let isKeyValue = false; + let i = 0; if (mapIter) - tabularData = previewEntries(tabularData); + [ tabularData, isKeyValue ] = previewEntries(tabularData); - if (mapIter || isMap(tabularData)) { + if (isKeyValue || isMap(tabularData)) { const keys = []; const values = []; let length = 0; - for (const [k, v] of tabularData) { - keys.push(inspect(k)); - values.push(inspect(v)); - length++; + if (mapIter) { + for (; i < tabularData.length / 2; ++i) { + keys.push(inspect(tabularData[i * 2])); + values.push(inspect(tabularData[i * 2 + 1])); + length++; + } + } else { + for (const [k, v] of tabularData) { + keys.push(inspect(k)); + values.push(inspect(v)); + length++; + } } return final([ iterKey, keyKey, valuesKey @@ -379,9 +389,9 @@ Console.prototype.table = function(tabularData, properties) { const setIter = isSetIterator(tabularData); if (setIter) - tabularData = previewEntries(tabularData); + [ tabularData ] = previewEntries(tabularData); - const setlike = setIter || isSet(tabularData); + const setlike = setIter || (mapIter && !isKeyValue) || isSet(tabularData); if (setlike) { const values = []; let length = 0; @@ -400,7 +410,7 @@ Console.prototype.table = function(tabularData, properties) { const valuesKeyArray = []; const indexKeyArray = ObjectKeys(tabularData); - for (var i = 0; i < indexKeyArray.length; i++) { + for (; i < indexKeyArray.length; i++) { const item = tabularData[indexKeyArray[i]]; const primitive = item === null || (typeof item !== 'function' && typeof item !== 'object'); diff --git a/lib/util.js b/lib/util.js index 15611fdae108df..8160d802b0332f 100644 --- a/lib/util.js +++ b/lib/util.js @@ -984,7 +984,7 @@ function formatMap(ctx, value, recurseTimes, keys) { function formatWeakSet(ctx, value, recurseTimes, keys) { const maxArrayLength = Math.max(ctx.maxArrayLength, 0); - const entries = previewEntries(value).slice(0, maxArrayLength + 1); + const [ entries ] = previewEntries(value).slice(0, maxArrayLength + 1); const maxLength = Math.min(maxArrayLength, entries.length); let output = new Array(maxLength); for (var i = 0; i < maxLength; ++i) @@ -1001,14 +1001,16 @@ function formatWeakSet(ctx, value, recurseTimes, keys) { function formatWeakMap(ctx, value, recurseTimes, keys) { const maxArrayLength = Math.max(ctx.maxArrayLength, 0); - const entries = previewEntries(value).slice(0, maxArrayLength + 1); - const remainder = entries.length > maxArrayLength; - const len = entries.length - (remainder ? 1 : 0); + const [ entries ] = previewEntries(value).slice(0, (maxArrayLength + 1) * 2); + // Entries exist as [key1, val1, key2, val2, ...] + const remainder = entries.length / 2 > maxArrayLength; + const len = entries.length / 2 - (remainder ? 1 : 0); const maxLength = Math.min(maxArrayLength, len); let output = new Array(maxLength); - for (var i = 0; i < len; i++) { - output[i] = `${formatValue(ctx, entries[i][0], recurseTimes)} => ` + - formatValue(ctx, entries[i][1], recurseTimes); + for (var i = 0; i < maxLength; i++) { + const pos = i * 2; + output[i] = `${formatValue(ctx, entries[pos], recurseTimes)} => ` + + formatValue(ctx, entries[pos + 1], recurseTimes); } // Sort all entries to have a halfway reliable output (if more entries than // retrieved ones exist, we can not reliably return the same output). @@ -1020,9 +1022,19 @@ function formatWeakMap(ctx, value, recurseTimes, keys) { return output; } +function zip2(list) { + const ret = Array(list.length / 2); + for (var i = 0; i < ret.length; ++i) + ret[i] = [list[2 * i], list[2 * i + 1]]; + return ret; +} + function formatCollectionIterator(ctx, value, recurseTimes, keys) { const output = []; - for (const entry of previewEntries(value)) { + var [ entries, isKeyValue ] = previewEntries(value); + if (isKeyValue) + entries = zip2(entries); + for (const entry of entries) { if (ctx.maxArrayLength === output.length) { output.push('... more items'); break; diff --git a/src/node_util.cc b/src/node_util.cc index 724bb3603cfddd..ef7dc8a818bf11 100644 --- a/src/node_util.cc +++ b/src/node_util.cc @@ -53,29 +53,16 @@ static void PreviewEntries(const FunctionCallbackInfo& args) { if (!args[0]->IsObject()) return; + Environment* env = Environment::GetCurrent(args); bool is_key_value; Local entries; if (!args[0].As()->PreviewEntries(&is_key_value).ToLocal(&entries)) return; - if (!is_key_value) - return args.GetReturnValue().Set(entries); - - uint32_t length = entries->Length(); - CHECK_EQ(length % 2, 0); - - Environment* env = Environment::GetCurrent(args); - Local context = env->context(); - - Local pairs = Array::New(env->isolate(), length / 2); - for (uint32_t i = 0; i < length / 2; i++) { - Local pair = Array::New(env->isolate(), 2); - pair->Set(context, 0, entries->Get(context, i * 2).ToLocalChecked()) - .FromJust(); - pair->Set(context, 1, entries->Get(context, i * 2 + 1).ToLocalChecked()) - .FromJust(); - pairs->Set(context, i, pair).FromJust(); - } - args.GetReturnValue().Set(pairs); + Local ret = Array::New(env->isolate(), 2); + ret->Set(env->context(), 0, entries).FromJust(); + ret->Set(env->context(), 1, v8::Boolean::New(env->isolate(), is_key_value)) + .FromJust(); + return args.GetReturnValue().Set(ret); } // Side effect-free stringification that will never throw exceptions. diff --git a/test/parallel/test-console-table.js b/test/parallel/test-console-table.js index 9ff4641d65923c..844d2e01e4180b 100644 --- a/test/parallel/test-console-table.js +++ b/test/parallel/test-console-table.js @@ -120,6 +120,26 @@ test(new Map([[1, 1], [2, 2], [3, 3]]).entries(), ` └───────────────────┴─────┴────────┘ `); +test(new Map([[1, 1], [2, 2], [3, 3]]).values(), ` +┌───────────────────┬────────┐ +│ (iteration index) │ Values │ +├───────────────────┼────────┤ +│ 0 │ 1 │ +│ 1 │ 2 │ +│ 2 │ 3 │ +└───────────────────┴────────┘ +`); + +test(new Map([[1, 1], [2, 2], [3, 3]]).keys(), ` +┌───────────────────┬────────┐ +│ (iteration index) │ Values │ +├───────────────────┼────────┤ +│ 0 │ 1 │ +│ 1 │ 2 │ +│ 2 │ 3 │ +└───────────────────┴────────┘ +`); + test(new Set([1, 2, 3]).values(), ` ┌───────────────────┬────────┐ │ (iteration index) │ Values │ diff --git a/test/parallel/test-util-inspect.js b/test/parallel/test-util-inspect.js index 874797665dc415..826e965fd0044d 100644 --- a/test/parallel/test-util-inspect.js +++ b/test/parallel/test-util-inspect.js @@ -447,13 +447,13 @@ assert.strictEqual(util.inspect(-5e-324), '-5e-324'); { const map = new Map(); map.set(1, 2); - const vals = previewEntries(map.entries()); + const [ vals ] = previewEntries(map.entries()); const valsOutput = []; for (const o of vals) { valsOutput.push(o); } - assert.strictEqual(util.inspect(valsOutput), '[ [ 1, 2 ] ]'); + assert.strictEqual(util.inspect(valsOutput), '[ 1, 2 ]'); } // Test for other constructors in different context.