Anzeige-Problem (Herunterskalierung)

[routeconverter]

Da du in BaseMapView#centerChanged nur den Cache für den aktuellen Zoom löschst, habe ich bei mir mal "reducedPositions.clear();" statt "reducedPositions.remove(getZoom());" eingebaut. Damit bekomme ich den Fehler auf die Schnelle nicht mehr hin.

Du hast recht: das war überoptimiert, nur den aktuellen Zoomlevel zu aktualisieren. Wenn man dann den Zoomlevel ändert und der Kartenausschnitt hat sich deutlich verändert, sind im Cache nur Positionen, die nicht mehr sichtbar sind. Commit

auch das zweite Problem habe ich gefunden - bzw. du hattest ja schon die richtige Ahnung.
Das passiert bei tiefen Zoomlevels, wenn der eine Punkt im real sichtbaren Bereich ist und der nächste Punkt ausserhalb des erweiterten Bereiches liegt.

Ich habe hier mal meine Lösung vom letzten Mal eingebaut, die dafür sorgt, dass immer der erste Punkt ausserhalb des Bereiches noch mit am Leben bleibt. Damit war dann der Fehler weg.

Das habe ich übernommen, die NullPointerException bei includeFirstAndLastPosition=false entfernt und einen Test gebaut. Schau mal auf den Commit

Ich werde dir dafür nochmal einen neuen Pull-Request schicken.
Wundere dich nicht über meinen Branchnamen. Ich hab nicht aufgebapsst und leider in meinem master-Branch alles gemacht.

Bitte mach Deine Pull-Requests klein wie möglich, dann haben sie eine bessere Chance angewendet zu werden: ich verstehe leichter, was die Änderung ist und kann in github einfach "Accept" klicken.

Was du über die Commit-Historie auch sehen wirst, ist dass ich einen Unittest für filterEveryNthPosition gemacht habe. Die Funktion hatte ich eigentlich zuerst im Verdacht, da sie auch nicht ganz korrekt gearbeitet hat.

Die Idee mit dem Test finde ich prima. Dabei habe ich auch herausgefunden, daß jeglicher Code, den wir da in #filterEveryNthPosition() bislang am Start hatten, nicht korrekt funktioniert hat. Schau mal auf den Commit