diff options
author | Trevor Norris <trev.norris@gmail.com> | 2015-06-02 20:09:03 +0300 |
---|---|---|
committer | Rod Vagg <rod@vagg.org> | 2015-08-04 21:56:11 +0300 |
commit | 36f78f4c1cbe0c0dd2a83f6b1dba534a9d722a49 (patch) | |
tree | 3709d5c5f0e3697e06c7f9ba3b56af51256aba84 /src/spawn_sync.cc | |
parent | 571ec13841699d255025b8eaeab17c84fdb4add3 (diff) |
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 79428fcd73c..97bccfd4b78 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; } |