Merge pull request #11 from ronaldsmartin/bugfix-api-17-concurrent-mo…

…dification-exception Fix API 17 ConcurrentModificationException
ronaldsmartin · Oct 15, 2016 · f40e4c9 · f40e4c9
2 parents 6089f9b + b1f7579
commit f40e4c9
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/material-viewpagerindicator/src/main/java/com/itsronald/widget/ViewPagerIndicator.java b/material-viewpagerindicator/src/main/java/com/itsronald/widget/ViewPagerIndicator.java
@@ -337,8 +337,8 @@ private void updateDotCount(int newDotCount) {
                 addViewInLayout(newDot, -1, layoutParams, true);
             }
         } else if (dotCount > newDotCount) {
-            final List<IndicatorDotView> removedDots = indicatorDots
-                    .subList(newDotCount, dotCount);
+            final List<IndicatorDotView> removedDots =
+                    new ArrayList<>(indicatorDots.subList(newDotCount, dotCount));
             for (IndicatorDotView removedDot : removedDots) {
                 removeViewInLayout(removedDot);
             }
@@ -370,8 +370,8 @@ private void updatePathCount(final int newPathCount) {
                 addViewInLayout(newPath, -1, layoutParams, true);
             }
         } else if (pathCount > newPathCount && newPathCount >= 0) {
-            final List<IndicatorDotPathView> pathsToRemove = dotPaths
-                    .subList(newPathCount, pathCount);
+            final List<IndicatorDotPathView> pathsToRemove =
+                    new ArrayList<>(dotPaths.subList(newPathCount, pathCount));
             for (IndicatorDotPathView dotPath : pathsToRemove) {
                 removeViewInLayout(dotPath);
             }