refactor: remove Ord bound from BinaryHeap::new etc

[aatifsyed]

This adds consistency with e.g BTreeMap::new, and makes it easier to e.g #[derive(Default)]¹

Footnotes

1. https://play.rust-lang.org/?version=stable&mode=debug&edition=2024&gist=f848e472a176fae155f17455bdfe0aee ↩

[rustbot]

r? @tgross35

rustbot has assigned @tgross35.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[tgross35]

This seems fine to me but will need libs-api FCP.

[aatifsyed]

Thanks! Is that this? https://rustc-dev-guide.rust-lang.org/stability.html#stabilizing-a-library-feature

What actions can I take to progress this :)

Sorry I've not made changes to std before!

[tgross35]

No action needed on your end :) the label I added means it will get discussed at this week’s meeting.

[Amanieu]

@rfcbot merge libs-api

[rust-rfcbot]

Team member @Amanieu has proposed to merge this. The next step is review by the rest of the tagged team members:

• @Amanieu
• @BurntSushi
• @dtolnay
• @joshtriplett
• @the8472

No concerns currently listed.

Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.