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

Update to Rust 2016-03-05 #9888

Merged
merged 1 commit into from Mar 6, 2016
Merged

Update to Rust 2016-03-05 #9888

merged 1 commit into from Mar 6, 2016

Conversation

@nox
Copy link
Member

nox commented Mar 6, 2016

@highfive
Copy link

highfive commented Mar 6, 2016

warning Warning warning

  • These commits modify net and script code, but no tests are modified. Please consider adding a test!
@nox
Copy link
Member Author

nox commented Mar 6, 2016

This is surprisingly way simpler than I expected it to be.

@@ -29,7 +29,7 @@ pub enum HeaderOrMethod {
impl HeaderOrMethod {
fn match_header(&self, header_name: &str) -> bool {
match *self {
HeaderOrMethod::HeaderData(ref s) => s.eq_ignore_ascii_case(header_name),
HeaderOrMethod::HeaderData(ref s) => (&**s).eq_ignore_ascii_case(header_name),

This comment has been minimized.

@Manishearth

Manishearth Mar 6, 2016

Member

why is autoderef not helping us here?

This comment has been minimized.

@nox nox force-pushed the rustup branch from 6b255c1 to 7c1dd54 Mar 6, 2016
@nox nox assigned Manishearth and unassigned asajeffrey Mar 6, 2016
@nox
Copy link
Member Author

nox commented Mar 6, 2016

@Manishearth Just bumped tenacious to 0.1.0 now you published it. r?

@Manishearth
Copy link
Member

Manishearth commented Mar 6, 2016

Reviewed 8 of 12 files at r1, 5 of 5 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

@Manishearth
Copy link
Member

Manishearth commented Mar 6, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 6, 2016

📌 Commit 7c1dd54 has been approved by Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Mar 6, 2016

Testing commit 7c1dd54 with merge 743e0c9...

bors-servo added a commit that referenced this pull request Mar 6, 2016
Update to Rust 2016-03-05

Blocked by Manishearth/rust-tenacious#14.

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

bors-servo commented Mar 6, 2016

@bors-servo bors-servo merged commit 7c1dd54 into master Mar 6, 2016
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@Ms2ger Ms2ger deleted the rustup branch Mar 6, 2016
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

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