diff options
| author | Trevor Norris <trev.norris@gmail.com> | 2015-07-22 13:06:54 -0600 |
|---|---|---|
| committer | Rod Vagg <rod@vagg.org> | 2015-08-04 11:56:16 -0700 |
| commit | e6ab2d92bc15c0f122ddc9310a9d69499008703a (patch) | |
| tree | 8eec8ddee104a2a25e0361fef3baec45bff0fb32 /src/node_buffer.cc | |
| parent | 9f727f5e03bbe6f798f0a084bf318f78f064013d (diff) | |
| download | android-node-v8-e6ab2d92bc15c0f122ddc9310a9d69499008703a.tar.gz android-node-v8-e6ab2d92bc15c0f122ddc9310a9d69499008703a.tar.bz2 android-node-v8-e6ab2d92bc15c0f122ddc9310a9d69499008703a.zip | |
buffer: fix not return on error
Throwing a JS error from C++ does not mean the function will return
early. This must be done manually.
Also remove extraneous comment no longer relevant.
Fix: 2903030 "buffer: switch API to return MaybeLocal<T>"
PR-URL: https://github.com/nodejs/io.js/pull/2225
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Diffstat (limited to 'src/node_buffer.cc')
| -rw-r--r-- | src/node_buffer.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/node_buffer.cc b/src/node_buffer.cc index b09f7333ff..5ab599ebab 100644 --- a/src/node_buffer.cc +++ b/src/node_buffer.cc @@ -287,7 +287,6 @@ MaybeLocal<Object> Copy(Isolate* isolate, const char* data, size_t length) { } -// Make a copy of "data". Why this isn't called "Copy", we'll never know. MaybeLocal<Object> New(Environment* env, const char* data, size_t length) { EscapableHandleScope scope(env->isolate()); @@ -474,7 +473,7 @@ void Slice(const FunctionCallbackInfo<Value>& args) { Maybe<bool> mb = ui->SetPrototype(env->context(), env->buffer_prototype_object()); if (!mb.FromMaybe(false)) - env->ThrowError("Unable to set Object prototype"); + return env->ThrowError("Unable to set Object prototype"); args.GetReturnValue().Set(ui); } |