GraphHopper.com | Forum | GitHub | Maps | Blog

Wrong usage of reverse argument in calcWeight


#1

Hi,

I think usage of reverse in this lines is wrong:

I read the usages of calcWeight and calcMillis and I think reverse is always false in non-bidirectional algorithms and is always false in calcMillis.

So we should use something like edgeState == edgeState.detach(true) instead of reverse to determine to use speed or reverseSpeed.