Commit 42507e6
src,node-api: update napi_is_detached_arraybuffer
This gets `napi_is_detached_arraybuffer(..)` to use the new API,
`ArrayBuffer::WasDetached`, that looks more straightforward.
Signed-off-by: Daeyeon Jeong <daeyeon.dev@gmail.com>
PR-URL: #45538
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>1 parent 81f63c2 commit 42507e6
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -3254,7 +3254,7 @@ napi_status NAPI_CDECL napi_is_detached_arraybuffer(napi_env env, | |||
| 3254 | 3254 | v8::Local<v8::Value> value = v8impl::V8LocalValueFromJsValue(arraybuffer); | |
| 3255 | 3255 | ||
| 3256 | 3256 | *result = | |
| 3257 | - value->IsArrayBuffer() && value.As<v8::ArrayBuffer>()->Data() == nullptr; | ||
| 3257 | + value->IsArrayBuffer() && value.As<v8::ArrayBuffer>()->WasDetached(); | ||
| 3258 | 3258 | ||
| 3259 | 3259 | return napi_clear_last_error(env); | |
| 3260 | 3260 | } | |
0 commit comments