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

crypto: move _impl call out of handleError funct

Browse Source 
This commit moves the _impl function call out of the handleError
function, which now only takes in an object as its parameter.

PR-URL: https://github.com/nodejs/node/pull/28318
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Daniel Bevenius 2019-06-20 08:09:17 +02:00
parent 7edd5f3bdd
commit c6f968de71
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/internal/crypto/keygen.js
View File

@ -60,7 +60,7 @@ function generateKeyPair(type, options, callback) {
callback.call(wrap, null, pubkey, privkey); callback.call(wrap, null, pubkey, privkey);
}; };
handleError(impl, wrap); handleError(impl(wrap));
} }
Object.defineProperty(generateKeyPair, customPromisifyArgs, { Object.defineProperty(generateKeyPair, customPromisifyArgs, {
@ -70,11 +70,10 @@ Object.defineProperty(generateKeyPair, customPromisifyArgs, {
function generateKeyPairSync(type, options) { function generateKeyPairSync(type, options) {
const impl = check(type, options); const impl = check(type, options);
return handleError(impl); return handleError(impl());
} }
function handleError(impl, wrap) { function handleError(ret) {
const ret = impl(wrap);
if (ret === undefined) if (ret === undefined)
return; // async return; // async