Skip to content
Permalink
Browse files

src: fix type name in comment

The comment refers to an exception type that JS land throws on the C++
code's behalf but apparently I changed the JS name before landing the
pull request and forgot to update the comment.

Refs: #20816

PR-URL: #28320
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
  • Loading branch information...
bnoordhuis authored and targos committed Jun 20, 2019
1 parent 19dad19 commit e334c1f13ba2885af9a9e6969c1e964d7065b043
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/node_crypto.cc
@@ -6085,7 +6085,7 @@ void Scrypt(const FunctionCallbackInfo<Value>& args) {
// EVP_PBE_scrypt() does not always put errors on the error stack
// and therefore ToResult() may or may not return an exception
// object. Return a sentinel value to inform JS land it should
// throw an ERR_CRYPTO_SCRYPT_PARAMETER_ERROR on our behalf.
// throw an ERR_CRYPTO_SCRYPT_INVALID_PARAMETER on our behalf.
auto result = job->ToResult();
if (result->IsUndefined()) result = Null(args.GetIsolate());
return args.GetReturnValue().Set(result);

0 comments on commit e334c1f

Please sign in to comment.
You can’t perform that action at this time.