From 341165cef44c7ecd67f3a911eb29e0433d33e14c Mon Sep 17 00:00:00 2001
From: Tobrun Van Nuland <tobrun.van.nuland@gmail.com>
Date: Thu, 17 Aug 2017 10:32:49 +0200
Subject: [android] - avoid IndexOutOfBounds when destroying map object

---
 .../mapbox/mapboxsdk/maps/AnnotationManager.java   | 46 +++++++++-------------
 1 file changed, 18 insertions(+), 28 deletions(-)

diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/AnnotationManager.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/AnnotationManager.java
index 712ab67de0..d15d5eddf8 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/AnnotationManager.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/AnnotationManager.java
@@ -212,22 +212,19 @@ class AnnotationManager {
       }
 
       if (markers.size() > 0) {
-        long[] ids = null;
+        long[] ids;
         if (nativeMapView != null) {
           ids = nativeMapView.addMarkers(markers);
+        } else {
+          ids = new long[markers.size()];
         }
 
-        long id = 0;
+        long id;
         Marker m;
-        for (int i = 0; i < markers.size(); i++) {
+        for (int i = 0; i < ids.length; i++) {
           m = markers.get(i);
           m.setMapboxMap(mapboxMap);
-          if (ids != null) {
-            id = ids[i];
-          } else {
-            // unit test
-            id++;
-          }
+          id = ids[i];
           m.setId(id);
           annotations.put(id, m);
         }
@@ -481,21 +478,18 @@ class AnnotationManager {
         }
       }
 
-      long[] ids = null;
+      long[] ids;
       if (nativeMapView != null) {
         ids = nativeMapView.addPolygons(polygons);
+      } else {
+        ids = new long[polygons.size()];
       }
 
-      long id = 0;
-      for (int i = 0; i < polygons.size(); i++) {
+      long id;
+      for (int i = 0; i < ids.length; i++) {
         polygon = polygons.get(i);
         polygon.setMapboxMap(mapboxMap);
-        if (ids != null) {
-          id = ids[i];
-        } else {
-          // unit test
-          id++;
-        }
+        id = ids[i];
         polygon.setId(id);
         shapeAnnotationIds.add(LAYER_ID_SHAPE_ANNOTATIONS + id);
         annotations.put(id, polygon);
@@ -555,23 +549,19 @@ class AnnotationManager {
         }
       }
 
-      long[] ids = null;
+      long[] ids;
       if (nativeMapView != null) {
         ids = nativeMapView.addPolylines(polylines);
+      } else {
+        ids = new long[polylines.size()];
       }
 
-      long id = 0;
+      long id;
       Polyline p;
-
-      for (int i = 0; i < polylines.size(); i++) {
+      for (int i = 0; i < ids.length; i++) {
         p = polylines.get(i);
         p.setMapboxMap(mapboxMap);
-        if (ids != null) {
-          id = ids[i];
-        } else {
-          // unit test
-          id++;
-        }
+        id = ids[i];
         p.setId(id);
         shapeAnnotationIds.add(LAYER_ID_SHAPE_ANNOTATIONS + id);
         annotations.put(id, p);
-- 
cgit v1.2.1