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

Upgrade Rust. #95

Merged
merged 1 commit into from Aug 1, 2014
Merged

Upgrade Rust. #95

merged 1 commit into from Aug 1, 2014

Conversation

@metajack
Copy link
Contributor

metajack commented Jul 31, 2014

No description provided.

glue.rs Outdated

pub fn RUST_INTERNED_STRING_TO_JSID(cx: *mut JSContext, str: *mut JSString) -> jsid;

pub fn DefineFunctionWithReserved(cx: *mut JSContext, obj: *mut JSObject,
name: *libc::c_char, call: JSNative, nargs: libc::c_uint,
name: *mut libc::c_char, call: JSNative, nargs: libc::c_uint,

This comment has been minimized.

@Ms2ger

Ms2ger Jul 31, 2014

Collaborator

Takes const char*

@metajack
Copy link
Contributor Author

metajack commented Jul 31, 2014

Fixed the const char thing.

Ms2ger added a commit that referenced this pull request Aug 1, 2014
@Ms2ger Ms2ger merged commit 2542bde into master Aug 1, 2014
@Ms2ger Ms2ger deleted the rustup-20140716 branch Aug 1, 2014
mmatyas pushed a commit to mmatyas/rust-mozjs that referenced this pull request Jul 30, 2015
Move some code out of unsafe blocks.
tschneidereit pushed a commit to tschneidereit/rust-mozjs that referenced this pull request Aug 26, 2017
…, r=Ms2ger

Expand upgrading instructions some more

* Mention building servo's bindgen fork
* Describe running `rust-mozjs/etc/bindings.sh` in more detail

r? @Ms2ger

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/mozjs/95)
<!-- Reviewable:end -->
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

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