karol/OpenTracks
1
0
Fork 0
forked from upstream-mirrors/OpenTracks
Code Pull Requests Actions Releases 2 Activity

GpsStatusManager should be running in on Thread alone.

Browse Source
This commit is contained in:
Dennis Guse
2023-08-09 19:58:18 +02:00
parent 4d5c59be96
commit f6e8a44a8b
1 changed files with 0 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/java/de/dennisguse/opentracks/services/handlers/GpsStatusManager.java
-11
View File
@@ -37,9 +37,6 @@ class GpsStatusManager {
@Nullable @Nullable
private TrackPoint lastTrackPoint = null; private TrackPoint lastTrackPoint = null;
// Flag to prevent GpsStatus checks two or more locations at the same time.
private boolean checking = false;
private Handler handler; private Handler handler;
public final Runnable gpsStatusTimer = () -> { public final Runnable gpsStatusTimer = () -> {
@@ -87,18 +84,10 @@ class GpsStatusManager {
* Receive new trackPoint and calculate the new status if needed. * 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. * 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) { public void onNewTrackPoint(@NonNull final TrackPoint trackPoint) {
if (checking) {
return;
}
checking = true;
lastTrackPoint = trackPoint; lastTrackPoint = trackPoint;
determineGpsStatusOnTrackpoint(trackPoint); determineGpsStatusOnTrackpoint(trackPoint);
checking = false;
} }
/** /**