Do not attempt to change map view if track is empty

andreynovikov · Feb 4, 2024 · 2d42615 · 2d42615
1 parent 169934a
commit 2d42615
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/app/src/main/java/mobi/maptrek/MainActivity.java b/app/src/main/java/mobi/maptrek/MainActivity.java
@@ -2750,6 +2750,8 @@ public void onTrackView(Track track) {
             updateLocationDrawable();
         }
         BoundingBox box = track.getBoundingBox();
+        if (box == null)
+            return;
         box.extendBy(0.05);
         mMap.animator().animateTo(box);
     }

diff --git a/app/src/main/java/mobi/maptrek/data/Track.java b/app/src/main/java/mobi/maptrek/data/Track.java
@@ -39,7 +39,8 @@ public class Track {
     private BoundingBox mBox = null;
 
     public BoundingBox getBoundingBox() {
-        //TODO Honor empty track
+        if (points.isEmpty())
+            return null;
         if (mBox == null) {
             mBox = new BoundingBox();
             synchronized (points) {
@@ -50,7 +51,7 @@ public BoundingBox getBoundingBox() {
         return mBox;
     }
 
-    public class TrackPoint extends GeoPoint {
+    public static class TrackPoint extends GeoPoint {
         public final boolean continuous;
         public final float elevation;
         public final float speed;