[android] - avoid creating InfoWindow iterator if no InfoWindows are …

…shown
mapbox · Jul 11, 2017 · c2bd866 · c2bd866
1 parent 90c61c1
commit c2bd866
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/...android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/InfoWindowManager.java b/...android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/InfoWindowManager.java
@@ -22,7 +22,8 @@
  */
 class InfoWindowManager {
 
-  private List<InfoWindow> infoWindows;
+  private final List<InfoWindow> infoWindows = new ArrayList<>();
+
   private MapboxMap.InfoWindowAdapter infoWindowAdapter;
   private boolean allowConcurrentMultipleInfoWindows;
 
@@ -31,12 +32,13 @@ class InfoWindowManager {
   private MapboxMap.OnInfoWindowCloseListener onInfoWindowCloseListener;
 
   InfoWindowManager() {
-    this.infoWindows = new ArrayList<>();
   }
 
   void update() {
-    for (InfoWindow infoWindow : infoWindows) {
-      infoWindow.update();
+    if (!infoWindows.isEmpty()) {
+      for (InfoWindow infoWindow : infoWindows) {
+        infoWindow.update();
+      }
     }
   }
 
@@ -56,10 +58,6 @@ boolean isAllowConcurrentMultipleOpenInfoWindows() {
     return allowConcurrentMultipleInfoWindows;
   }
 
-  List<InfoWindow> getInfoWindows() {
-    return infoWindows;
-  }
-
   boolean isInfoWindowValidForMarker(@NonNull Marker marker) {
     return !TextUtils.isEmpty(marker.getTitle()) || !TextUtils.isEmpty(marker.getSnippet());
   }