Repository navigation
Closed
Labels
A-binaryheapArea: `std::collections::BinaryHeap`Area: `std::collections::BinaryHeap`B-unstableBlocker: Implemented in the nightly compiler and unstable.Blocker: Implemented in the nightly compiler and unstable.C-tracking-issueCategory: An issue tracking the progress of sth. like the implementation of an RFCCategory: An issue tracking the progress of sth. like the implementation of an RFCI-libs-radarLibs issues that are tracked on the team's radar.Libs issues that are tracked on the team's radar.Libs-SmallLibs issues that are considered "small" or self-containedLibs issues that are considered "small" or self-containedT-libs-api[DEPRECATED; DO NOT USE][DEPRECATED; DO NOT USE]disposition-mergeThis issue / PR is in PFCP or FCP with a disposition to merge it.This issue / PR is in PFCP or FCP with a disposition to merge it.finished-final-comment-periodThe final comment period is finished for this PR / Issue.The final comment period is finished for this PR / Issue.
Description
This is a tracking issue for BinaryHeap::retain.
The feature gate for the issue is #![feature(binary_heap_retain)].
Added in PR #71485
Metadata
Metadata
Assignees
Labels
A-binaryheapArea: `std::collections::BinaryHeap`Area: `std::collections::BinaryHeap`B-unstableBlocker: Implemented in the nightly compiler and unstable.Blocker: Implemented in the nightly compiler and unstable.C-tracking-issueCategory: An issue tracking the progress of sth. like the implementation of an RFCCategory: An issue tracking the progress of sth. like the implementation of an RFCI-libs-radarLibs issues that are tracked on the team's radar.Libs issues that are tracked on the team's radar.Libs-SmallLibs issues that are considered "small" or self-containedLibs issues that are considered "small" or self-containedT-libs-api[DEPRECATED; DO NOT USE][DEPRECATED; DO NOT USE]disposition-mergeThis issue / PR is in PFCP or FCP with a disposition to merge it.This issue / PR is in PFCP or FCP with a disposition to merge it.finished-final-comment-periodThe final comment period is finished for this PR / Issue.The final comment period is finished for this PR / Issue.
Activity
KodrAus commentedon Oct 30, 2020
We discussed this at the recent Libs meeting and had a question about taking
&Trather than&mut T. In the end we thought keeping consistent withVecand not having to think about resorting afterwards made the&Titem type appropriate.If anybody is keen to open a stabilization PR we can kick off a FCP!
Rollup merge of rust-lang#78681 - m-ou-se:binary-heap-retain, r=Amanieu
Auto merge of rust-lang#78681 - m-ou-se:binary-heap-retain, r=Amanieu
JohnTitor commentedon Jul 21, 2022
@rust-lang/libs-api May I ask to kick off a FCP? Stabilizing it will unblock rust-lang/rust-clippy#9059.
Amanieu commentedon Jul 21, 2022
@rfcbot fcp merge
rfcbot commentedon Jul 21, 2022
Team member @Amanieu has proposed to merge this. The next step is review by the rest of the tagged team members:
Concerns:
&mutresolved by Tracking Issue for binary heap retain #71503 (comment)predicate panic can leave heap in invalid stateresolved by Tracking Issue for binary heap retain #71503 (comment)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.
22 remaining items
rfcbot commentedon Mar 14, 2023
🔔 This is now entering its final comment period, as per the review above. 🔔
rfcbot commentedon Mar 24, 2023
The final comment period, with a disposition to merge, as per the review above, is now complete.
As the automated representative of the governance process, I would like to thank the author for their work and everyone else who contributed.
This will be merged soon.
Stabilize `binary_heap_retain`
Auto merge of rust-lang#109701 - Amanieu:binaryheap_retain, r=ChrisDe…
Rollup merge of #106918 - dtolnay:heapretain, r=the8472