Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

Commit

Permalink
fix: microsoft-goto warning w/ clang-cl
Browse files Browse the repository at this point in the history
  • Loading branch information
nornagon committed Sep 19, 2018
1 parent f26e949 commit fd1a701
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/node.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2432,13 +2432,15 @@ static void DebugProcess(const FunctionCallbackInfo<Value>& args) {
wchar_t mapping_name[32];
LPTHREAD_START_ROUTINE* handler = nullptr;

DWORD pid;

if (args.Length() != 1) {
env->ThrowError("Invalid number of arguments.");
goto out;
}

CHECK(args[0]->IsNumber());
DWORD pid = args[0].As<Integer>()->Value();
pid = args[0].As<Integer>()->Value();

process = OpenProcess(PROCESS_CREATE_THREAD | PROCESS_QUERY_INFORMATION |
PROCESS_VM_OPERATION | PROCESS_VM_WRITE |
Expand Down

0 comments on commit fd1a701

Please sign in to comment.