Can this ever run more than two times?

Do we need the for loop?

I used a loop because the original implementation was recursive — I assumed /*/ could appear multiple times, so I kept the behavior but replaced recursion with a loop

@srpvpn I hear you, it makes sense.

Can you please find out if this needs to be done once, twice or many more times please? I'd rather remove the for loop unless necessary.

@VojtechVitek
ReplaceAll doesn’t handle overlapping patterns — for example, /// becomes /*/, so we need to run it multiple times.
Since paths can have several wildcards in a row, a loop ensures they're all collapsed. The test with three wildcards confirms this.

Sorry. My question was if if it ever overlaps more than two times, since you introduced ReplaceAll()?

@VojtechVitek
Docs say “ReplaceAll returns a copy of the string s with all non-overlapping instances of old replaced by new”.
Because the scan resumes after the piece it just wrote, any overlap is skipped:

/// --1st pass--> /*/ --2nd pass--> /

So if a router tacks on three-plus /* segments, one call isn’t enough.