Wayname calculation logic recalculates line slice unnecessarily often

[avi-c]

The Android calculation of the current wayname was originally ported directly from the iOS logic.

mapbox/mapbox-navigation-ios#2635 tracks a performance improvement for this functionality that can be very significant for maps with many polyline segments. The Android algorithm should fix this issue as well.

I believe the corresponding code is here:

mapbox-navigation-android/libnavigation-ui/src/main/java/com/mapbox/navigation/ui/map/WaynameFeatureFilter.java

Line 99 in cc93c3d

LineString stepSliceFromCurrentPoint = lineSlice(currentPoint, lastStepPoint, currentStepLineString);

The calculation of this LineSlice should be done outside of the double for-loop since it doesn't change.

[1ec5]

/cc @abhishek1508

[Guardiola31337]

Where are we here? It seems a pretty straightforward fix if not already addressed.

cc @cafesilencio @abhishek1508

[Guardiola31337]

Any updates here? @cafesilencio @abhishek1508

[cafesilencio]

I can't find this class or similar code in 2.x. I"m going to close this since our current efforts are on 2.x.