diff options
| author | Trevor Norris <trev.norris@gmail.com> | 2015-06-02 11:09:03 -0600 |
|---|---|---|
| committer | Rod Vagg <rod@vagg.org> | 2015-08-04 11:56:11 -0700 |
| commit | 36f78f4c1cbe0c0dd2a83f6b1dba534a9d722a49 (patch) | |
| tree | 3709d5c5f0e3697e06c7f9ba3b56af51256aba84 /src/spawn_sync.cc | |
| parent | 571ec13841699d255025b8eaeab17c84fdb4add3 (diff) | |
| download | android-node-v8-36f78f4c1cbe0c0dd2a83f6b1dba534a9d722a49.tar.gz android-node-v8-36f78f4c1cbe0c0dd2a83f6b1dba534a9d722a49.tar.bz2 android-node-v8-36f78f4c1cbe0c0dd2a83f6b1dba534a9d722a49.zip | |
buffer: switch API to return MaybeLocal<T>
Instead of aborting in case of internal failure, return an empty
Local<Object>. Using the MaybeLocal<T> API, users must check their
return values.
PR-URL: https://github.com/nodejs/io.js/pull/1825
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Diffstat (limited to 'src/spawn_sync.cc')
| -rw-r--r-- | src/spawn_sync.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/spawn_sync.cc b/src/spawn_sync.cc index 79428fcd73..97bccfd4b7 100644 --- a/src/spawn_sync.cc +++ b/src/spawn_sync.cc @@ -167,7 +167,7 @@ void SyncProcessStdioPipe::Close() { Local<Object> SyncProcessStdioPipe::GetOutputAsBuffer(Environment* env) const { size_t length = OutputLength(); - Local<Object> js_buffer = Buffer::New(env, length); + Local<Object> js_buffer = Buffer::New(env, length).ToLocalChecked(); CopyOutput(Buffer::Data(js_buffer)); return js_buffer; } |