Conversation
|
It's rare that @rust-highfive didn't show up 😮 |
src/libcollections/str.rs
Outdated
There was a problem hiding this comment.
Insta-stable? AFAICT there’s no tracking issue which went through FCP for stabilisation?
There was a problem hiding this comment.
@nagisa I'm sorry... Should I submit a RFC about this? BTW, where can I know the process of accepting a new method?
There was a problem hiding this comment.
An unstable method with a corresponding tracking issue may be enough.
There was a problem hiding this comment.
@nagisa Am I supposed to open the tracking issue by myself?
There was a problem hiding this comment.
I’d suggest to only do that after you get thumbs-up from the libs team for the feature you’re adding. In the mean-time you can use 0 as a placeholder issue for the attribute.
|
This PR needs a reviewer, @rust-lang/libs. |
e89edb9 to
f89f3a0
Compare
|
@nagisa Thanks for your advice! 😄 |
|
Seems quite niche to me. |
f89f3a0 to
1b65a57
Compare
src/libcollections/str.rs
Outdated
There was a problem hiding this comment.
Using .take(count) would be the more typical way to limit the number of iterations here. (I'd understand it right away :D)
1b65a57 to
2b95aab
Compare
|
Discussed at libs triage today the decision was to merge. @knight42 could you open an issue on rust-lang/rust to track the stabilization of this and update the |
2b95aab to
ebda770
Compare
|
@alexcrichton Great to hear that! 🎉 The |
|
@bors: r+ Thanks! |
|
📌 Commit ebda770 has been approved by |
|
⌛ Testing commit ebda770 with merge e25e32c... |
|
💔 Test failed - auto-win-msvc-64-opt-rustbuild |
|
@alexcrichton the failure seems spurious |
|
@bors: retry On Tue, Sep 13, 2016 at 8:59 PM, Jian Zeng notifications@github.com wrote:
|
Implement std::str::replacen
Replaces first N matches of a pattern with another string.
```
assert_eq!("acaaa".replacen(a, "b", 3), "bcbba")
```
Replaces first N matches of a pattern with another string.