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

core::slice cleanup #15288

Merged
merged 2 commits into from
Jul 1, 2014
Merged

core::slice cleanup #15288

merged 2 commits into from
Jul 1, 2014

Conversation

brson
Copy link
Contributor

@brson brson commented Jun 30, 2014

This does two things:

  • Reorganizes the declaration order to be more readable, less random.
  • Removes the slice::traits module, a public module that does nothing but declare impls of cmp traits.

brson added 2 commits June 30, 2014 15:40
This is just moving things around so they are defined in a logical order.
This is only breaking for code that references the empty `slice::traits`
module.

[breaking-change]
bors added a commit that referenced this pull request Jul 1, 2014
This does two things:

* Reorganizes the declaration order to be more readable, less random.
* Removes the `slice::traits` module, a public module that does nothing but declare impls of `cmp` traits.
@bors bors closed this Jul 1, 2014
@bors bors merged commit fa5bc6f into rust-lang:master Jul 1, 2014
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 17, 2023
Handle TyAlias in projected_ty

First of all I still have no idea how MIR works but rust-lang#15143 has been an issue that constantly made RA crash so I have been looking for a way to make RA stop panicking. I have zero claims that what I want to merge has any sense or is correct 😄  but there isn't any more panicking. Even if it is wrong may this be at least a step towards resolving this issue.
As is customary this PR fixes rust-lang#15143
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