diff options
author | Rich Trott <rtrott@gmail.com> | 2022-01-17 08:31:21 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-01-17 08:31:21 +0300 |
commit | a7215c8fa7b0f6950e2e0e0c8431b0454521721f (patch) | |
tree | 0a4f25f225fa7ce95d33cdcf703a3c4d73b8ba6b /benchmark | |
parent | 986cf3b986c6c3e1495327eb3aa1ffc3972f2442 (diff) |
benchmark: remove unreachable code from crypto/hash-stream-creation
`hash.digest('buffer')` has returned a Buffer and not a string since at
least Node.js 0.10.6. The benchmark, as it is written, will not work on
any version of Node.js prior to 16.x (due to `Object.hasOwn()`) and
certainly won't run on versions earlier than 0.10.6 due to const/let and
probably other things. Remove impossible-to-reach code intended to
accommodate Node.js earlier than 0.10.6.
PR-URL: https://github.com/nodejs/node/pull/41535
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Diffstat (limited to 'benchmark')
-rw-r--r-- | benchmark/crypto/hash-stream-creation.js | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/benchmark/crypto/hash-stream-creation.js b/benchmark/crypto/hash-stream-creation.js index c21eb6eaaae..e480c7b6edc 100644 --- a/benchmark/crypto/hash-stream-creation.js +++ b/benchmark/crypto/hash-stream-creation.js @@ -52,11 +52,7 @@ function legacyWrite(algo, message, encoding, writes, len, outEnc) { while (writes-- > 0) { const h = crypto.createHash(algo); h.update(message, encoding); - let res = h.digest(outEnc); - - // Include buffer creation costs for older versions - if (outEnc === 'buffer' && typeof res === 'string') - res = Buffer.from(res, 'binary'); + h.digest(outEnc); } bench.end(gbits); |