Skip to content

Commit

Permalink
src: move node_process to modern THROW_ERR*
Browse files Browse the repository at this point in the history
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #35472
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
  • Loading branch information
jasnell authored and MylesBorins committed Oct 14, 2020
1 parent 4164477 commit 541082c
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions src/node_process_methods.cc
Original file line number Diff line number Diff line change
Expand Up @@ -164,8 +164,9 @@ static void Kill(const FunctionCallbackInfo<Value>& args) {
Environment* env = Environment::GetCurrent(args);
Local<Context> context = env->context();

if (args.Length() != 2)
return env->ThrowError("Bad argument.");
if (args.Length() < 2) {
THROW_ERR_MISSING_ARGS(env, "Bad argument.");
}

int pid;
if (!args[0]->Int32Value(context).To(&pid)) return;
Expand Down Expand Up @@ -319,8 +320,8 @@ static void ResourceUsage(const FunctionCallbackInfo<Value>& args) {
static void DebugProcess(const FunctionCallbackInfo<Value>& args) {
Environment* env = Environment::GetCurrent(args);

if (args.Length() != 1) {
return env->ThrowError("Invalid number of arguments.");
if (args.Length() < 1) {
return THROW_ERR_MISSING_ARGS(env, "Invalid number of arguments.");
}

CHECK(args[0]->IsNumber());
Expand All @@ -344,9 +345,8 @@ static void DebugProcess(const FunctionCallbackInfo<Value>& args) {
Environment* env = Environment::GetCurrent(args);
Isolate* isolate = args.GetIsolate();

if (args.Length() != 1) {
env->ThrowError("Invalid number of arguments.");
return;
if (args.Length() < 1) {
return THROW_ERR_MISSING_ARGS(env, "Invalid number of arguments.");
}

HANDLE process = nullptr;
Expand Down

0 comments on commit 541082c

Please sign in to comment.