Tracking issue for `str::replacen`

[knight42]

Implemented in #36347

cc @alexcrichton

[alexcrichton]

@rfcbot fcp merge

Still looks like a good API to me!

[rfcbot]

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

• @BurntSushi
• @Kimundi
• @alexcrichton
• @aturon
• @brson
• @sfackler

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.

[brson]

What's special abut the first N of a thing during replacement that it needs its own function? I don't see much discussion on the linked PR that created this, and I don't recall anything about it.

[knight42]

@brson
I often come across a case that I only need to replace the first match in a line during text processing. Since str::replace will always replace all the matches, I guess adding a parameter to control the count of occurrences that it replaces would be more convenient.

[aturon]

Note that this method follows the convention set out by methods like splitn.

I suspect replacen with a count of 1 will be a common usage.

[knight42]

@brson Does this sound good to you? Or are there any improvement that I can make?

[brson]

@knight42 it's good. Thanks.

[aturon]

Manually moving to final comment period with disposition to stabilize.

[rfcbot]

🔔 This is now entering its final comment period, as per the review above. 🔔

psst @alexcrichton, I wasn't able to add the final-comment-period label, please do so.