Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v11.14.0 proposal #27163

Merged
merged 133 commits into from
Apr 11, 2019
Merged
Changes from 1 commit
Commits
Show all changes
133 commits
Select commit Hold shift + click to select a range
62f9049
src: refactor node options parsers to mitigate MSVC bug
refack Feb 23, 2019
e6c1ad5
src: fix warnings around node_options
refack Feb 25, 2019
63e7cc7
src: forbid access to CLI options before bootstrapping is done
joyeecheung Mar 6, 2019
ab70c96
src: refactor coverage connection
joyeecheung Mar 8, 2019
03bd649
test: show stderr on v8 coverage test failures
joyeecheung Mar 8, 2019
d4f9509
process: delay process.argv[0] and process.argv0 handling
joyeecheung Mar 8, 2019
75c5d9c
process: create legacy process properties during pre-execution
joyeecheung Mar 8, 2019
5a29a94
http: reduce usage of public util
ZYSzys Mar 9, 2019
c51cc9e
src: apply clang-tidy rule modernize-make-unique
gengjiawen Mar 7, 2019
7671a65
embedding: refactor public `ArrayBufferAllocator` API
addaleax Mar 8, 2019
5e64acd
embedding: make `NewIsolate()` API more flexible
addaleax Mar 8, 2019
1b45704
inspector: patch C++ debug options instead of process._breakFirstLine
joyeecheung Mar 11, 2019
169f3f7
util: require `isNativeError` from internalBinding
ZYSzys Mar 19, 2019
21486e5
util: extract uncurryThis function for reuse
ZYSzys Mar 19, 2019
4defe47
child_process: reduce internal usage of public require of util
toshi1127 Mar 19, 2019
44450ef
worker: remove usage of require('util')
toshi1127 Mar 20, 2019
b24e45a
timers: deprecate active() and _unrefActive()
Fishrock123 Mar 18, 2019
5d9f819
worker: remove usage of require('util') in worker_thread.js
toshi1127 Mar 20, 2019
bb9f1cc
fs: reduce usage of require('util')
toshi1127 Mar 25, 2019
a03552d
process: handle --expose-internals during pre-execution
joyeecheung Mar 18, 2019
e43dbaa
console: remove unreachable code
Trott Mar 25, 2019
80e845e
test: add test about unencrypted PKCS#8 private key for RSA
sasurau4 Mar 25, 2019
c1def07
src: delete unreachable code in heap_utils.cc
gengjiawen Mar 20, 2019
ea7e2c0
src: tidy up include headers in env.cc
gengjiawen Mar 20, 2019
c824127
src: delete unused field in class ModuleWrap
gengjiawen Mar 20, 2019
c969731
src: use deleted function for class BaseObject
gengjiawen Mar 20, 2019
6bfb17f
src: fix task release in cares_wrap.cc
gengjiawen Mar 20, 2019
0427354
src: delete useless code in cares_wrap.cc
gengjiawen Mar 20, 2019
a1cf745
test: refactor path parse test
BridgeAR Mar 26, 2019
bd006e1
path: fix win32 parse regression
BridgeAR Mar 26, 2019
1b08e62
util: improve proxy inspection
BridgeAR Mar 26, 2019
682b410
worker: allow execArgv and eval in combination
addaleax Mar 8, 2019
c756b84
errors: make range mandatory in ERR_OUT_OF_RANGE
BridgeAR Mar 26, 2019
17bff5c
doc: remove reference to "credentials object"
sam-github Mar 25, 2019
a821a96
doc: correct typo: cert.issuerCertificate
srl295 Mar 26, 2019
98552f3
crypto: allow undefined for saltLength and padding
tniessen Mar 26, 2019
de93737
benchmark: remove unused field in class BenchmarkProgress
gengjiawen Mar 26, 2019
25d5198
doc: change links to https in benchmark guide
gengjiawen Mar 26, 2019
5512ecb
benchmark: tidy up eslint ignore in foreach-bench.js
gengjiawen Mar 26, 2019
2ef1bd9
test: do not require flags when executing a file
BridgeAR Mar 22, 2019
edad9af
test: move hasCrypto check
BridgeAR Mar 27, 2019
fd6381b
test: remove test-path-parse-6229.js from known issues
BridgeAR Mar 26, 2019
1aa6e99
util: fix map entries inspection
BridgeAR Mar 26, 2019
b62739c
path: remove dead code
BridgeAR Mar 26, 2019
d3c1de3
dns: remove dns.promises experimental warning
cjihrig Mar 11, 2019
ac12109
dns: make dns.promises enumerable
cjihrig Mar 27, 2019
4a7a84a
doc: move dns.promises to stable status
cjihrig Mar 27, 2019
d76c30c
doc: add note about mkdtemp() platform differences
cjihrig Mar 27, 2019
9330d7e
test: add known_issues test for fs.copyFile()
Trott Mar 27, 2019
769d12c
build: add a `Prepare ccache` job in Travis
richardlau Mar 29, 2019
6cbd6b5
build: fix skipping of flaky tests on Travis
richardlau Mar 28, 2019
e776b01
src: do not call into JS in the maxAsyncCallStackDepthChanged interrupt
joyeecheung Mar 26, 2019
760d089
inspector: display error when ToggleAsyncHook fails
joyeecheung Mar 22, 2019
28efecc
doc: remove "How is an LTS release cut?" section
Trott Mar 28, 2019
8798db3
url: add ws: and wss: to slashedProtocol set
lpinca Mar 27, 2019
17cc117
doc: update changelog for v10.x LTS
BethGriggs Mar 26, 2019
0f615d4
util: add subclass and null prototype support for errors in inspect
BridgeAR Mar 26, 2019
0aea4d1
benchmark,lib: change var to const
BridgeAR Mar 26, 2019
7fe43bd
build: remove unused label from vcbuild.bat
bnoordhuis Mar 25, 2019
dc9ce86
tools: update dependencies in lint-md-cli-rollup
watilde Mar 24, 2019
e5181f8
test: simplify for loop in test-buffer-zero-fill-cli.js
juanarbol Mar 20, 2019
78fad32
readline: replace quadratic regex with linear one
Hakerh400 Mar 19, 2019
d3840bc
src: allow per-Environment set of env vars
addaleax Mar 13, 2019
729e2f2
src: implement generic backend for process.env
addaleax Mar 13, 2019
8bd7909
worker: use copy of process.env
addaleax Feb 22, 2019
3c92926
module: simpler esm loading
BridgeAR Mar 29, 2019
ea47189
stream: do not unconditionally call `_read()` on `resume()`
addaleax Mar 28, 2019
b64b223
build,win: silence MSVC warning C4129 for V8
refack Mar 30, 2019
2396743
child_process: doc deprecate ChildProcess._channel
cjihrig Mar 29, 2019
7a39196
doc: change code lang and update it with latest Node.js
gengjiawen Mar 29, 2019
fbdead7
src: add missing uv_fs_req_cleanup()
cjihrig Mar 30, 2019
06dce39
test: fix error code typo
cjihrig Mar 31, 2019
d0ee1a3
src: fix warning on mismatched fn signature
sam-github Mar 27, 2019
8258f07
test: remove unused triggerAsyncId param in test
juanarbol Mar 20, 2019
7cbe121
tools: update ESLint to 5.16.0
cjihrig Mar 30, 2019
57207e7
doc: list when promiseResolve hook was added to async_hooks
watson Mar 29, 2019
0b94aec
doc: add information about modules cache behavior
BridgeAR Mar 28, 2019
64a74d5
doc: move "Prints: ..." under the code
simon300000 Mar 31, 2019
96e429a
dns: refactor lib/internal/dns/utils.js
Trott Mar 30, 2019
a0b0e42
test: add IPv6 brackets but no port to test-dns
Trott Mar 30, 2019
b766c06
build,meta: tweak Travis config
refack Mar 28, 2019
5102d69
src: remove internal includes from node_crypto.h
sam-github Mar 28, 2019
4c96ef7
doc: improve the doc of the 'information' event
lpinca Mar 30, 2019
687348a
test: move test that creates 1Gb file to pummel
Trott Apr 2, 2019
ccd3b74
fs: remove experimental warning for fs.promises
addaleax Mar 11, 2019
5945cf0
tools: fix `test.py --time`
richardlau Mar 30, 2019
38da9be
test: remove test-trace-events-api-worker-disabled from flaky
Trott Mar 30, 2019
054cf7b
src: replace c-style cast
gengjiawen Mar 24, 2019
5fdee11
doc: remove description duplication in buffer.md
vsemozhetbyt Mar 30, 2019
fd3b031
doc: fix possible typo in buffer.md
vsemozhetbyt Mar 31, 2019
944ce7b
doc: add missing types in buffer.md
vsemozhetbyt Mar 31, 2019
5fd4053
doc: unify number/integer types in buffer.md
vsemozhetbyt Mar 31, 2019
3a35d8a
doc: fix error notes in `Buffer.from()` variants
vsemozhetbyt Mar 30, 2019
f9ef9eb
doc: add note about Buffer octets integer coercion
vsemozhetbyt Mar 31, 2019
ea75d43
doc: mark optional parameters in buffer.md
vsemozhetbyt Mar 31, 2019
19ccee0
doc: add notes about negative offsets in buffer.md
vsemozhetbyt Mar 31, 2019
221bcfb
doc: unify periods in comments in buffer.md
vsemozhetbyt Mar 30, 2019
00eef26
doc: unify link formatting in buffer.md
vsemozhetbyt Mar 31, 2019
e40bfd8
util: `inspect()` should not exceed `breakLength`
BridgeAR Mar 26, 2019
a318cbc
src: elevate v8::Task namespace
juanarbol Mar 21, 2019
addc73a
path: refactor for less indentation
BridgeAR Mar 26, 2019
316372c
doc: remove old system_errors
JungMinu Mar 29, 2019
fec9f76
process: run RunBootstrapping in CreateEnvironment
joyeecheung Mar 19, 2019
d3c1481
report: add cwd to report
cjihrig Mar 31, 2019
0fc5dcc
doc: remove unnecessary intro in governance doc
Trott Mar 31, 2019
26af6fb
doc: document the 'pause' and 'resume' events
lpinca Mar 29, 2019
0ac41fc
stream: make Symbol.asyncIterator support stable
mcollina Mar 29, 2019
6d39460
readline: make Symbol.asyncIterator support stable
mcollina Apr 1, 2019
58da879
module: fix repl require calling the same file again
BridgeAR Mar 26, 2019
3fce303
test: make module test pass with NODE_PENDING_DEPRECATION
addaleax Mar 30, 2019
3e4c03d
deps: update nghttp2 to 1.37.0
gengjiawen Mar 29, 2019
438bc52
src: apply clang-tidy rule bugprone-incorrect-roundings
gengjiawen Mar 24, 2019
1bbe5b3
process: store argv in Environment
joyeecheung Mar 19, 2019
a58437d
build: only emit download ICU warnings once
richardlau Mar 31, 2019
7547d8e
doc: fix default maxBuffer size
koh110 Sep 17, 2018
2f1da92
meta: move ofrobots to TSC emeritus
ofrobots Apr 3, 2019
a21bee7
src: use sizeof(var) instead of sizeof(type)
bnoordhuis Apr 1, 2019
c62b1a2
benchmark: add new module loading benchmarks
BridgeAR Jan 2, 2019
4c7a2f1
benchmark: improve module-loader benchmark
BridgeAR Mar 28, 2019
cdb0d84
module: inline try catch
BridgeAR Mar 28, 2019
8f6921e
module: add path to the module object
BridgeAR Mar 28, 2019
5917f4d
module: add extra caching layer
BridgeAR Mar 28, 2019
36714e1
buffer: fix concat error message
BridgeAR Apr 2, 2019
cee079f
deps: add ARM64 Windows support in openssl
shigeki Feb 23, 2019
ddf7477
deps: add ARM64 Windows configurations in openssl
jkunkee Mar 8, 2019
038b778
querystring: simplify stringify method
ZYSzys Mar 27, 2019
70d9f32
src: prevent crash in TTYWrap::Initialize
Hakerh400 Mar 21, 2019
066cd60
crypto: fix crash of encrypted private key export without cipher
panva Apr 1, 2019
d688a93
async_hooks: minor cleanup and improvements
apapirovski Mar 31, 2019
b502873
build: fix inspector dependency resolution
bnoordhuis Mar 31, 2019
a00d840
crypto: fix crash of encrypted private key export without cipher
panva Apr 1, 2019
75e8377
doc: assign missed deprecation code
richardlau Apr 9, 2019
cd026f8
2019-04-11, Version 11.14.0 (Current)
BethGriggs Apr 9, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
console: remove unreachable code
In the current code, line 497 checks if `item` is `null` or `undefined`.
However, `item` is guaranteed to be a non-null object or function at
that point.

* Lines 484/485 set `primitive` to `true` if `item` is null or
  undefined.
* Line 486 skips line 497 if `primitive` is true (which it will always
  be if `item` is null or undefined) and `properties` is undefined. So
  the only way to get to line 497 when `item` is null or undefined is if
  `properties` is specified.
* Line 494 skips line 497 if `primitive` is true (which it will always
  be if `item` is null or undefined) and `properties` are specified
  (which will always be the case or else this `else` block is skipped.)

Here are the current lines 484 through 497:

      const primitive = item === null ||
          (typeof item !== 'function' && typeof item !== 'object');
      if (properties === undefined && primitive) {
        hasPrimitives = true;
        valuesKeyArray[i] = _inspect(item);
      } else {
        const keys = properties || ObjectKeys(item);
        for (const key of keys) {
          if (map[key] === undefined)
            map[key] = [];
          if ((primitive && properties) || !hasOwnProperty(item, key))
            map[key][i] = '';
          else
            map[key][i] = item == null ? item : _inspect(item[key]);

This change removes the unnecessary ternary in that final line,
simplifying it to:

            map[key][i] = _inspect(item[key]);

PR-URL: #26906
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
  • Loading branch information
Trott authored and targos committed Mar 30, 2019
commit e43dbaaba4ae44d35898794284c73985df5889c5
2 changes: 1 addition & 1 deletion lib/internal/console/constructor.js
Original file line number Diff line number Diff line change
Expand Up @@ -492,7 +492,7 @@ const consoleMethods = {
if ((primitive && properties) || !hasOwnProperty(item, key))
map[key][i] = '';
else
map[key][i] = item == null ? item : _inspect(item[key]);
map[key][i] = _inspect(item[key]);
}
}
}
Expand Down