GetNext should have no side effect when failed
This commit is contained in:
parent
b6c353ee80
commit
7160054927
1 changed files with 5 additions and 2 deletions
|
@ -36,8 +36,11 @@ class Arguments {
|
|||
insufficient_arguments_ = true;
|
||||
return false;
|
||||
}
|
||||
v8::Local<v8::Value> val = (*info_)[next_++];
|
||||
return ConvertFromV8(isolate_, val, out);
|
||||
v8::Local<v8::Value> val = (*info_)[next_];
|
||||
bool success = ConvertFromV8(isolate_, val, out);
|
||||
if (success)
|
||||
next_++;
|
||||
return success;
|
||||
}
|
||||
|
||||
template<typename T>
|
||||
|
|
Loading…
Reference in a new issue