Open-Harbor/osrm-backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 3 Wiki Activity

Fix unnecessary duplication of backward segments for non-split edges

Browse Source 
for edges with the split flag false and in_backward_direction true
if in_forward_direction is also true backward segments can be incorrectly duplicated
This commit is contained in:
Michael Krasnyk 2017-03-29 10:10:57 +02:00 committed by Patrick Niklaus
parent 7a1a209168
commit 4ab3165ae3
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/extractor/extractor_callbacks.cpp
View File

@ -330,7 +330,7 @@ void ExtractorCallbacks::ProcessWay(const osmium::Way &input_way, const Extracti
(turn_lane_id_forward != turn_lane_id_backward)); (turn_lane_id_forward != turn_lane_id_backward));
if (in_forward_direction) if (in_forward_direction)
{ { // add (forward) segments or (forward,backward) for non-split edges in backward direction
util::for_each_pair( util::for_each_pair(
nodes.cbegin(), nodes.cbegin(),
nodes.cend(), nodes.cend(),
@ -355,8 +355,8 @@ void ExtractorCallbacks::ProcessWay(const osmium::Way &input_way, const Extracti
}); });
} }
if (in_backward_direction || split_edge) if (in_backward_direction && (!in_forward_direction || split_edge))
{ { // add (backward) segments for split edges or not in forward direction
util::for_each_pair( util::for_each_pair(
nodes.cbegin(), nodes.cbegin(),
nodes.cend(), nodes.cend(),