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

Run on stable Rust #14

Merged
merged 7 commits into from
Jul 24, 2015
Merged

Run on stable Rust #14

merged 7 commits into from
Jul 24, 2015

Conversation

SimonSapin
Copy link
Member

Without the unstable Cargo feature, on 64 bit platforms:

  • Tendril<_> is 17 bytes instead of 16. (It has a drop flag.)
  • Option<Tendril<_>> is 18 bytes instead of 16. (It has a drop flag and an explicit tag.)

r? @Manishearth

Needed for servo/html5ever#53

Review on Reviewable

This avoids breaking dependent crates when e.g. Rust deprecates something.
Only use `unsafe_no_drop_flag` and `NonZero` when the `unstable` Cargo feature is enabled.

Without the feature, on 64 bit platforms:
* `Tendril<_>` is 17 bytes instead of 16. (It has a drop flag.)
* `Option<Tendril<_>>` is 18 bytes instead of 16. (It has a drop flag and an explicit tag.)
@SimonSapin
Copy link
Member Author

Blocked on reem/rust-debug-unreachable#2 being on crates.io

SimonSapin added a commit to SimonSapin/travis-doc-upload that referenced this pull request Jul 19, 2015
This will avoid duplicate uploads when Travis-CI runs with multiple Rust version like in servo/tendril#14
@mbrubeck
Copy link

Reviewed 3 of 7 files at r1, 6 of 6 files at r2.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from the review on Reviewable.io

@mbrubeck
Copy link

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit b4d2c2f has been approved by mbrubeck

@bors-servo
Copy link
Contributor

⌛ Testing commit b4d2c2f with merge faf97de...

bors-servo pushed a commit that referenced this pull request Jul 24, 2015
Run on stable Rust

Without the `unstable` Cargo feature, on 64 bit platforms:
* `Tendril<_>` is 17 bytes instead of 16. (It has a drop flag.)
* `Option<Tendril<_>>` is 18 bytes instead of 16. (It has a drop flag and an explicit tag.)

r? @Manishearth

Needed for servo/html5ever#53

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

☀️ Test successful - travis

@bors-servo bors-servo merged commit b4d2c2f into master Jul 24, 2015
@SimonSapin SimonSapin deleted the stable branch July 24, 2015 21:07
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