Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replicate CallArgs::{get, index, index_mut} onto JSJitMethodCallArgs #223

Merged
merged 1 commit into from Dec 7, 2015

Conversation

@KiChjang
Copy link
Member

KiChjang commented Dec 6, 2015

This solves the problem of servo/servo#8854 where I can't use .index() on some args values because the type of args is JSJitMethodCallArgs instead of CallArgs.

Review on Reviewable

@KiChjang KiChjang force-pushed the KiChjang:js-jit-method-ca branch from 459eeb2 to ac7e773 Dec 6, 2015
@frewsxcv
Copy link
Member

frewsxcv commented Dec 6, 2015

/cc @Ms2ger

@Ms2ger
Copy link
Collaborator

Ms2ger commented Dec 7, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Dec 7, 2015

📌 Commit ac7e773 has been approved by Ms2ger

@bors-servo
Copy link
Contributor

bors-servo commented Dec 7, 2015

Testing commit ac7e773 with merge 07523d8...

bors-servo added a commit that referenced this pull request Dec 7, 2015
Replicate CallArgs::{get, index, index_mut} onto JSJitMethodCallArgs

This solves the problem of servo/servo#8854 where I can't use `.index()` on some args values because the type of args is `JSJitMethodCallArgs` instead of `CallArgs`.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/rust-mozjs/223)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 7, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit ac7e773 into servo:master Dec 7, 2015
1 check was pending
1 check was pending
homu Testing commit ac7e773 with merge 07523d8...
Details
@KiChjang KiChjang deleted the KiChjang:js-jit-method-ca branch Apr 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.