Commit 4b8b5fe
src: use MaybeLocal.ToLocal instead of IsEmpty
PR-URL: #35716
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>1 parent 01d7c46 commit 4b8b5fe
1 file changed
Lines changed: 4 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -424,12 +424,13 @@ MaybeLocal<Object> New(Environment* env, | |||
| 424 | 424 | True(env->isolate())).IsNothing()) { | |
| 425 | 425 | return Local<Object>(); | |
| 426 | 426 | } | |
| 427 | - MaybeLocal<Uint8Array> ui = Buffer::New(env, ab, 0, length); | ||
| 427 | + MaybeLocal<Uint8Array> maybe_ui = Buffer::New(env, ab, 0, length); | ||
| 428 | 428 | ||
| 429 | - if (ui.IsEmpty()) | ||
| 429 | + Local<Uint8Array> ui; | ||
| 430 | + if (!maybe_ui.ToLocal(&ui)) | ||
| 430 | 431 | return MaybeLocal<Object>(); | |
| 431 | 432 | ||
| 432 | - return scope.Escape(ui.ToLocalChecked()); | ||
| 433 | + return scope.Escape(ui); | ||
| 433 | 434 | } | |
| 434 | 435 | ||
| 435 | 436 | // Warning: This function needs `data` to be allocated with malloc() and not | |
0 commit comments