Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upTracking issue for VecDeque Extras #27788
Comments
Gankro
added
the
B-unstable
label
Aug 13, 2015
aturon
added
the
T-libs
label
Aug 13, 2015
This comment has been minimized.
This comment has been minimized.
|
|
This comment has been minimized.
This comment has been minimized.
|
I think we can pretty easily stabilize |
sfackler
added
the
I-nominated
label
Sep 17, 2015
This comment has been minimized.
This comment has been minimized.
|
This issue is now entering its cycle-long FCP for stabilization in 1.5 Specifically the |
alexcrichton
added
final-comment-period
and removed
I-nominated
labels
Sep 24, 2015
steveklabnik
added this to the 1.5 milestone
Oct 1, 2015
This comment has been minimized.
This comment has been minimized.
|
@alexcrichton How should the swapping methods be named? |
This comment has been minimized.
This comment has been minimized.
|
@llogiq as mentioned in the issue description, the other candidate names would be |
This comment has been minimized.
This comment has been minimized.
|
The libs team discussed this during triage today and the decision was to stabilize:
|
This comment has been minimized.
This comment has been minimized.
|
Oh, we forgot to discuss as_mut_slices vs as_slices_mut. I think we should move muts to the back as much as possible, even if in this case we're making "mutable slices", it's still the _mut variant of as_slices. |
This comment has been minimized.
This comment has been minimized.
|
Hm I'm not sure I'd follow the same reasoning, in method names types like |
This comment has been minimized.
This comment has been minimized.
|
Yeah I've been a bit ambivalent about that naming scheme, but I guess it's On Thu, Oct 22, 2015 at 11:18 AM, Alex Crichton notifications@github.com
|
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Oct 23, 2015
bors
added a commit
that referenced
this issue
Oct 24, 2015
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Oct 24, 2015
bors
added a commit
that referenced
this issue
Oct 25, 2015
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Oct 25, 2015
bors
added a commit
that referenced
this issue
Oct 25, 2015
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Oct 25, 2015
bors
added a commit
that referenced
this issue
Oct 25, 2015
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Oct 25, 2015
bors
added a commit
that referenced
this issue
Oct 25, 2015
arcnmx
added a commit
to arcnmx/rust
that referenced
this issue
Nov 2, 2015
alexcrichton
removed
the
final-comment-period
label
Nov 5, 2015
This comment has been minimized.
This comment has been minimized.
|
The only remaining unstable methods on @rfcbot fcp merge |
This comment has been minimized.
This comment has been minimized.
rfcbot
commented
Jan 19, 2017
•
|
Team member @sfackler has proposed to merge this. The next step is review by the rest of the tagged teams: No concerns currently listed. Once these reviewers reach consensus, 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. |
This comment has been minimized.
This comment has been minimized.
rfcbot
commented
Jan 23, 2017
|
|
Gankro commentedAug 13, 2015
This covers the
deque_extrasblanket feature, which should probably be sharded more in the future. It covers:truncateresizeas_slicesas_mut_slicesswap_back_removeswap_front_removeSome notes:
truncateandresizeare strange because they aren't directional (they just truncate off the back). Need work.as_slicesandas_mut_slicesseem good to go. A bit niche, but so is VecDeque. Maybe move themutto the end. RFC'd and everything.swap_back_removeandswap_front_removealso seem good to go. Maybe move the directional specifier to the end. RFC'd and everything.