diff options
| author | Vladimir Byko-Ianko <v.bykoianko@corp.mail.ru> | 2016-08-12 10:26:20 +0300 |
|---|---|---|
| committer | Vladimir Byko-Ianko <v.bykoianko@corp.mail.ru> | 2016-08-19 10:24:07 +0300 |
| commit | 98996a1567bba771225b96c5df9d3d7bd0424556 (patch) | |
| tree | 081d2038d2198627c5425c7a20d7456c0cb26b0b /routing | |
| parent | 80ae9e54d04fe437bbe54df0e10a3502a39af3b6 (diff) | |
Bugfix. Crash on zero length routes.
Diffstat (limited to 'routing')
| -rw-r--r-- | routing/routing_session.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/routing/routing_session.cpp b/routing/routing_session.cpp index 720f22996d..0de247706d 100644 --- a/routing/routing_session.cpp +++ b/routing/routing_session.cpp @@ -323,13 +323,13 @@ void RoutingSession::GetRouteFollowingInfo(FollowingInfo & info) const double RoutingSession::GetCompletionPercent() const { - // Uncreated route can't have completion percents. - if (!m_route.IsValid()) + double const denominator = m_passedDistanceOnRouteMeters + m_route.GetTotalDistanceMeters(); + if (!m_route.IsValid() || denominator == 0.0) return 0; double const percent = 100.0 * (m_passedDistanceOnRouteMeters + m_route.GetCurrentDistanceFromBeginMeters()) / - (m_passedDistanceOnRouteMeters + m_route.GetTotalDistanceMeters()); + denominator; if (percent - m_lastCompletionPercent > kCompletionPercentAccuracy) { auto const lastGoodPoint = |