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

Place collections into the root namespace of collection crate #15637

Closed
imv opened this issue Jul 12, 2014 · 1 comment
Closed

Place collections into the root namespace of collection crate #15637

imv opened this issue Jul 12, 2014 · 1 comment

Comments

@imv
Copy link

imv commented Jul 12, 2014

Currently fully-qualified names of collections contain the name of the collection twice, e.g. collections::btree::BTree. This extra btree does not appear to serve any purpose. There would be no name collisions if all collections are placed directly into the root namespace, as all functionality is implemented as methods. The sub-modules could be left for things like collections::dlist::Items, whereas collections::dlist::DList should become just collections::DList.

@bluss
Copy link
Member

bluss commented Jul 12, 2014

This is already the case. (By the way collections as a crate is a subpart of libstd, and thus you should use it through std::collections if you can.)

Doc links:

@huonw huonw closed this as completed Jul 12, 2014
bors added a commit to rust-lang-ci/rust that referenced this issue Nov 13, 2023
Update chalk version

This only pulls in one commit in Chalk, which is an improvement to inference for rust-analyzer: rust-lang/chalk#800
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

No branches or pull requests

3 participants