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

Fix E0303 explanation. #27013

Merged
merged 1 commit into from
Jul 17, 2015
Merged

Fix E0303 explanation. #27013

merged 1 commit into from
Jul 17, 2015

Conversation

michaelsproul
Copy link
Contributor

A merge in #24523 broke the explanation for E0303. This commit restores the previous version and also removes an erroneous & (which had nothing to do with the merge).

@rust-highfive
Copy link
Collaborator

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

@bors: r+ 4ec3ab6 rollup

Thanks!

@bors
Copy link
Contributor

bors commented Jul 14, 2015

⌛ Testing commit 4ec3ab6 with merge 2e514af...

@bors
Copy link
Contributor

bors commented Jul 14, 2015

💔 Test failed - auto-mac-64-nopt-t

@michaelsproul
Copy link
Contributor Author

The failure, in std::sys::process::tests::test_process_mask has nothing to do with my patch. Retry?

@alexcrichton
Copy link
Member

@bors: retry

On Tue, Jul 14, 2015 at 12:07 AM, Michael Sproul notifications@github.com
wrote:

The failure, in std::sys::process::tests::test_process_mask has nothing
to do with my patch. Retry?


Reply to this email directly or view it on GitHub
#27013 (comment).

Manishearth added a commit to Manishearth/rust that referenced this pull request Jul 16, 2015
…hton

 A merge in rust-lang#24523  broke the explanation for E0303. This commit restores the previous version and also removes an erroneous `&` (which had nothing to do with the merge).
@bors bors merged commit 4ec3ab6 into rust-lang:master Jul 17, 2015
@michaelsproul michaelsproul deleted the fix-E0303 branch July 17, 2015 08:06
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.

4 participants