Skip to content

Conversation

redfire75369
Copy link
Contributor

Removing the two wrapped array buffer functions from the bindings also shortens imports (mozjs::jsapi::<function> as compared to mozjs::jsapi::glue::<function>).

@redfire75369 redfire75369 force-pushed the array-buffer/unique-ptr branch 2 times, most recently from fc73188 to 5595fe8 Compare July 29, 2024 06:00
@jdm jdm enabled auto-merge July 29, 2024 06:14
@jdm jdm added this pull request to the merge queue Jul 29, 2024
@sagudev sagudev removed this pull request from the merge queue due to a manual request Jul 29, 2024
@sagudev
Copy link
Member

sagudev commented Jul 29, 2024

This will need rebase after #483 lands, so let's wait for it to land.

Added Array Buffer Functions using `js::UniquePtr` to Bindgen Blacklist

Signed-off-by: Redfire <[email protected]>
@sagudev sagudev force-pushed the array-buffer/unique-ptr branch from 5595fe8 to fe0c843 Compare July 29, 2024 06:35
@redfire75369 redfire75369 force-pushed the array-buffer/unique-ptr branch from fe0c843 to d687d67 Compare July 29, 2024 06:36
@redfire75369
Copy link
Contributor Author

Rebased on latest, with version bump accordingly.

@sagudev sagudev enabled auto-merge July 29, 2024 06:36
@sagudev sagudev added this pull request to the merge queue Jul 29, 2024
Merged via the queue into servo:main with commit 5fb90b6 Jul 29, 2024
@redfire75369 redfire75369 deleted the array-buffer/unique-ptr branch July 29, 2024 07:29
jschwe pushed a commit to jschwe/mozjs that referenced this pull request Aug 2, 2024
* Added Wrapper for NewArrayBufferWithContents

Added Array Buffer Functions using `js::UniquePtr` to Bindgen Blacklist

Signed-off-by: Redfire <[email protected]>

* Bumped mozjs-sys Version

Signed-off-by: Redfire <[email protected]>

---------

Signed-off-by: Redfire <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants