Skip to content

Add use a::b::{c, mod}; to the manual #16537

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

Merged
merged 1 commit into from Aug 17, 2014
Merged

Add use a::b::{c, mod}; to the manual #16537

merged 1 commit into from Aug 17, 2014

Conversation

ghost
Copy link

@ghost ghost commented Aug 16, 2014

No description provided.


An example of `use` declarations:

~~~~
use std::iter::range_step;
use std::option::{Some, None};
use std::collections::hash::{mod, HashMap};
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be std::collections::hashmap::{mod, HashMap} unless I'm missing something.

Copy link
Author

Choose a reason for hiding this comment

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

Of course! Thank you.

@steveklabnik
Copy link
Member

Thanks for adding this! I would like another reviewer to confirm the grammar is correct, though.

@bors bors closed this Aug 17, 2014
@bors bors merged commit 7606f58 into rust-lang:master Aug 17, 2014
@ghost ghost deleted the use-mod-manual branch August 18, 2014 18:19
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 18, 2024
internal: tool discovery prefers sysroot tools

Fixes rust-lang/rust-analyzer#15927, Fixes rust-lang/rust-analyzer#16523

After this PR we will look for `cargo` and `rustc` in the sysroot if it was succesfully loaded instead of using the current lookup scheme. This should be more correct than the current approach as that relies on the working directory of the server binary or loade workspace, meaning it can behave a bit odd wrt overrides.

Additionally, rust-project.json projects now get the target data layout set so there should be better const eval support now.
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