Skip to content
This repository has been archived by the owner on Nov 12, 2022. It is now read-only.

Accept a mutable handle in JS_GetPromiseResult. #470

Merged
merged 1 commit into from
Aug 31, 2019
Merged

Conversation

jdm
Copy link
Member

@jdm jdm commented Aug 30, 2019

This makes it easier to use from calling code that needs to root the result.


This change is Reviewable

@jdm
Copy link
Member Author

jdm commented Aug 30, 2019

r? @asajeffrey

@asajeffrey
Copy link
Member

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 9effecf has been approved by asajeffrey

@bors-servo
Copy link
Contributor

⌛ Testing commit 9effecf with merge 92136c0...

bors-servo pushed a commit that referenced this pull request Aug 30, 2019
Accept a mutable handle in JS_GetPromiseResult.

This makes it easier to use from calling code that needs to root the result.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-mozjs/470)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - checks-travis, status-appveyor
Approved by: asajeffrey
Pushing 92136c0 to master...

@bors-servo bors-servo merged commit 9effecf into master Aug 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants