From 028587657bb5b0ee40fa1d00c308f7b4c3650dee Mon Sep 17 00:00:00 2001 From: Omer Munam <39843323+omer-munam@users.noreply.github.com> Date: Sun, 11 Feb 2024 00:07:13 +0000 Subject: [PATCH] Fixing nested if issue in TrackStatisticsUpdater --- .../opentracks/stats/TrackStatisticsUpdater.java | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/main/java/de/dennisguse/opentracks/stats/TrackStatisticsUpdater.java b/src/main/java/de/dennisguse/opentracks/stats/TrackStatisticsUpdater.java index 31dc37936..a2300079d 100644 --- a/src/main/java/de/dennisguse/opentracks/stats/TrackStatisticsUpdater.java +++ b/src/main/java/de/dennisguse/opentracks/stats/TrackStatisticsUpdater.java @@ -141,11 +141,8 @@ public void addTrackPoint(TrackPoint trackPoint) { currentSegment.addTotalDistance(movingDistance); } - if (!currentSegment.isIdle() && !trackPoint.isSegmentManualStart()) { - if (lastTrackPoint != null) { - currentSegment.addMovingTime(trackPoint, lastTrackPoint); - } - } + if (!currentSegment.isIdle() && !trackPoint.isSegmentManualStart() && lastTrackPoint != null) { + currentSegment.addMovingTime(trackPoint, lastTrackPoint); if (trackPoint.getType() == TrackPoint.Type.IDLE) { currentSegment.setIdle(true);