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

TrackDataHub: do not send more TrackPoints after stop().

Browse Source
This commit is contained in:
Dennis Guse
2021-05-24 16:56:20 +02:00
parent 7929d219ac
commit ccfcd3b306
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/java/de/dennisguse/opentracks/content/TrackDataHub.java
+7 -5
View File
@@ -342,14 +342,16 @@ public class TrackDataHub {
} }
TrackPoint trackPoint = null; TrackPoint trackPoint = null;
TrackStatisticsUpdater currentUpdater = trackStatisticsUpdater; //TODO This prevents a NPE if stop() is happening while notifyTrackPointsTableUpdate()
if (currentUpdater == null) {
Log.e(TAG, "notifyTrackPointsTableUpdate() while being stopped.");
return;
}
try (TrackPointIterator trackPointIterator = contentProviderUtils.getTrackPointLocationIterator(selectedTrackId, next)) { try (TrackPointIterator trackPointIterator = contentProviderUtils.getTrackPointLocationIterator(selectedTrackId, next)) {
while (trackPointIterator.hasNext()) { while (trackPointIterator.hasNext()) {
if (!isStarted()) {
break;
}
//Prevents a NPE if stop() is happening while notifyTrackPointsTableUpdate()
TrackStatisticsUpdater currentUpdater = trackStatisticsUpdater;
trackPoint = trackPointIterator.next(); trackPoint = trackPointIterator.next();
TrackPoint.Id trackPointId = trackPoint.getId(); TrackPoint.Id trackPointId = trackPoint.getId();