diff options
| author | Constantin Shalnev <c.shalnev@corp.mail.ru> | 2015-06-23 18:56:54 +0300 |
|---|---|---|
| committer | Alex Zolotarev <alex@maps.me> | 2015-09-23 02:52:27 +0300 |
| commit | c0a5b3be8873f1abc02d93b6e334241fb83132d2 (patch) | |
| tree | c18fcdbcd4d18cf4954ecda01e7c34be4a0f359f /routing/road_graph.cpp | |
| parent | 6ed6bb77775a3e47d884af60224f4611f937ee86 (diff) | |
Optimization: removed redundant copying and calculations
Diffstat (limited to 'routing/road_graph.cpp')
| -rw-r--r-- | routing/road_graph.cpp | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/routing/road_graph.cpp b/routing/road_graph.cpp index 1241d51506..a8c01dd522 100644 --- a/routing/road_graph.cpp +++ b/routing/road_graph.cpp @@ -165,9 +165,6 @@ IRoadGraph::CrossEdgesLoader::CrossEdgesLoader(m2::PointD const & cross, TEdgeVe void IRoadGraph::CrossEdgesLoader::operator()(uint32_t featureId, RoadInfo const & roadInfo) { - if (roadInfo.m_points.empty()) - return; - size_t const numPoints = roadInfo.m_points.size(); for (size_t i = 0; i < numPoints; ++i) @@ -182,8 +179,7 @@ void IRoadGraph::CrossEdgesLoader::operator()(uint32_t featureId, RoadInfo const // p // o------------>o - Edge edge(featureId, false /* forward */, i - 1, p, roadInfo.m_points[i - 1]); - m_outgoingEdges.push_back(edge); + m_outgoingEdges.emplace_back(featureId, false /* forward */, i - 1, p, roadInfo.m_points[i - 1]); } if (i < numPoints - 1) @@ -191,8 +187,7 @@ void IRoadGraph::CrossEdgesLoader::operator()(uint32_t featureId, RoadInfo const // p // o------------>o - Edge edge(featureId, true /* forward */, i, p, roadInfo.m_points[i + 1]); - m_outgoingEdges.push_back(edge); + m_outgoingEdges.emplace_back(featureId, true /* forward */, i, p, roadInfo.m_points[i + 1]); } } } |