Skip to content

RUST_LOG=::help should be sorted #8949

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
huonw opened this issue Sep 3, 2013 · 3 comments · Fixed by #11236
Closed

RUST_LOG=::help should be sorted #8949

huonw opened this issue Sep 3, 2013 · 3 comments · Fixed by #11236
Labels
A-diagnostics Area: Messages for errors, warnings, and lints

Comments

@huonw
Copy link
Member

huonw commented Sep 3, 2013

It makes it much easier to find what one is looking for.

$ RUST_LOG=::help hello-world

Crate log map:

 std::io
 std::path
 std::os
 std::condition
 std::ptr

It's a lot worse for e.g. RUST_LOG=::help rustc, since it has a much longer list of modules.

@alexcrichton
Copy link
Member

This will probably be super easy once #8880 lands.

cc #3309

@fhahn
Copy link
Contributor

fhahn commented Sep 4, 2013

I had a look at this issue, but it seems like all sorting functions live in extra::sort, which cannot be imported from stdlib.

How should this be tackled? I think the sorting functions were moved out of std to extra after 0.6, but I didn't follow that discussion. Is there a way to reuse the existing sort functions?

@huonw
Copy link
Member Author

huonw commented Sep 4, 2013

Doesn't look like it, although presumably there will be sorting in std in the future; so I'd suggest either get a sort in std now, or just wait. (Part of the problem is extra::sort is very slow: extra::priority_queue::PriorityQueue::from_vec(v).to_sorted_vec() is actually faster than the dedicated sorting algorithms.)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-diagnostics Area: Messages for errors, warnings, and lints
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants