foster/denoland-rusty-v8
0
0
Fork 0
mirror of https://github.com/denoland/rusty_v8.git synced 2025-01-11 08:34:01 -05:00
Code Issues Activity

refactor: Have ArrayBuffer::data return Option<NonNull<c_void>> (#1131)

Browse source 
The pointer returned by `ArrayBuffer::data` might be null if the
backing store has zero length, but the return type `*mut c_void` does
not force the user to consider this case. This change makes the return
type `Option<NonNull<c_void>>`, which is semantically equivalent, but
which forces users of the API to handle the `None` case.

This PR is the `ArrayBuffer` counterpart to #817.

This is a breaking API change.
This commit is contained in:
Andreu Botella 2022-11-24 15:31:11 +01:00 committed by GitHub
parent 3a05ab4cd6
commit d718370a07
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/array_buffer.rs
View file

@ -431,8 +431,9 @@ impl ArrayBuffer {
/// More efficient shortcut for GetBackingStore()->Data().
/// The returned pointer is valid as long as the ArrayBuffer is alive.
#[inline(always)]
pub fn data(&self) -> *mut c_void {
unsafe { v8__ArrayBuffer__Data(self) }
pub fn data(&self) -> Option<NonNull<c_void>> {
let raw_ptr = unsafe { v8__ArrayBuffer__Data(self) };
NonNull::new(raw_ptr)
}
/// Get a shared pointer to the backing store of this array buffer. This

8
tests/test_api.rs
View file

@ -7275,8 +7275,14 @@ fn backing_store_data() {
let store = v8::ArrayBuffer::new_backing_store_from_vec(v).make_shared();
let buf = v8::ArrayBuffer::with_backing_store(&mut scope, &store);
assert_eq!(buf.byte_length(), len);
assert!(buf.data().is_some());
assert_eq!(
unsafe { std::slice::from_raw_parts_mut(buf.data() as *mut u8, len) },
unsafe {
std::slice::from_raw_parts_mut(
buf.data().unwrap().cast::<u8>().as_ptr(),
len,
)
},
&[1, 2, 3, 4, 5]
);
}