From f6e8a44a8b21532162186839c6adb2063743f679 Mon Sep 17 00:00:00 2001
From: Dennis Guse <dennis.guse@alumni.tu-berlin.de>
Date: Wed, 9 Aug 2023 19:58:18 +0200
Subject: [PATCH] GpsStatusManager should be running in on Thread alone.

---
 .../services/handlers/GpsStatusManager.java           | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/src/main/java/de/dennisguse/opentracks/services/handlers/GpsStatusManager.java b/src/main/java/de/dennisguse/opentracks/services/handlers/GpsStatusManager.java
index 4875641fc..121814783 100644
--- a/src/main/java/de/dennisguse/opentracks/services/handlers/GpsStatusManager.java
+++ b/src/main/java/de/dennisguse/opentracks/services/handlers/GpsStatusManager.java
@@ -37,9 +37,6 @@ class GpsStatusManager {
     @Nullable
     private TrackPoint lastTrackPoint = null;
 
-    // Flag to prevent GpsStatus checks two or more locations at the same time.
-    private boolean checking = false;
-
     private Handler handler;
 
     public final Runnable gpsStatusTimer = () -> {
@@ -87,18 +84,10 @@ class GpsStatusManager {
      * Receive new trackPoint and calculate the new status if needed.
      * It look for GPS changes in lastLocation if it's not null. If it's null then look for in lastValidLocation if any.
      */
-    //TODO Remove checking; should be synchronized if this is a problem.
     public void onNewTrackPoint(@NonNull final TrackPoint trackPoint) {
-        if (checking) {
-            return;
-        }
-
-        checking = true;
         lastTrackPoint = trackPoint;
 
         determineGpsStatusOnTrackpoint(trackPoint);
-
-        checking = false;
     }
 
     /**