Fix `goto_tele` not finding all teleporters

[ChillerDragon]

I noticed that on the current master the goto_tele feature was skipping one tele.

https://github.com/ddnet/ddnet/assets/20344300/afface81-7199-4419-be2a-9e80d8ada79b

I did not want to debug the code because it looked so complex. So I refactored it and it also fixed the bug. As you can see it now finds one more tele in the same map:

https://github.com/ddnet/ddnet/assets/20344300/49dc9ed3-d1f6-43db-a80a-9ca71d86b6b2

Also the performance should be much better. Since now the worst case is only iterating as many times as there are teleporters of that type in the map. Before in the worst case it would iterate over every single tile in the tele layer (including empty tiles).

Checklist

• [x] Tested the change ingame
• [x] Provided screenshots if it is a visual change
• [ ] Tested in combination with possibly related configuration options
• [ ] Written a unit test (especially base/) or added coverage to integration test
• [ ] Considered possible null pointers and out of bounds array indexing
• [ ] Changed no physics that affect existing maps
• [ ] Tested the change with ASan+UBSan or valgrind's memcheck (optional)

[heinrich5991]

I find it hard to see which tele got added in the "new" video.

The old code tried to ensure that teles that are jumped to are a bit apart, is that why it missed one?

[ChillerDragon]

I find it hard to see which tele got added in the "new" video.

The old code tried to ensure that teles that are jumped to are a bit apart, is that why it missed one?

Those were both missing. It still does the distance check. But it did never come near this area in the old code.