I keep iterating because a tracker url we're renaming to may exist further on the list. We'll end up with two or more identical trackers then, especially if we remove ` && !match` from

```c++
if (tracker_url == QUrl(misc::toQString(it->url)) && !match) {
```
This is highly unlikely, but can happen.
I just don't know if users should be able to shoot themselves in the leg.