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

Removed @ from around several rustc contexts. #12735

Merged
merged 17 commits into from
Mar 17, 2014

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Mar 6, 2014

No description provided.

@eddyb
Copy link
Member Author

eddyb commented Mar 6, 2014

I was trying to hint at something else with the branch name, but the coincidence is amazing.

@eddyb
Copy link
Member Author

eddyb commented Mar 6, 2014

@alexcrichton I think I did that to avoid adding a lifetime parameter to ty::ctxt.
But I like the idea that creating a type context requires ownership over the Session.
You can always recover it by destroying the type context (which I do in one or two places in that commit).

@alexcrichton
Copy link
Member

That sounds good to me. I think this is just waiting on the previous change to land first now.

None => self.default_sysroot.as_ref()
.expect("missing sysroot and default_sysroot in Session")
};
filesearch::FileSearch::new(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this expensive? because it seems that this is now created several times, rather than just once.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's just a few references.

@bors bors closed this Mar 17, 2014
@bors bors merged commit e2ebc8f into rust-lang:master Mar 17, 2014
@eddyb eddyb deleted the at-exodus-chapter-11 branch March 17, 2014 11:07
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
fix(completion): `super::` completion at crate root and module depth aware

- should close rust-lang#12439
- Suggest super according to the depth in tree
flip1995 pushed a commit to flip1995/rust that referenced this pull request May 2, 2024
Changelog for Clippy 1.78 🪄

Roses and Violets have colors,
Red and Blue are the two,
I'm getting to the end of my masters,
what a cool goal to pursue

---

### The cat of this release is: *Shadow* submitted by `@benwh1:`

<img height=500 src="https://github.com/rust-lang/rust-clippy/assets/32911992/c56af314-4644-482a-a08e-f32f4c7d7b22" alt="The cats of this Clippy release" />

Cats for the next release can be nominated in the comments :D

---

changelog: none
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.

5 participants