Merge pull request #89 from what3words/fix/concurrent-modification-exception-when-adding-marker

Fix ConcurrentModificationException in addMarker method

Author: phpduy

lib-compose/src/main/java/com/what3words/components/compose/maps/W3WMapManager.kt

@@ -1475,19 +1475,21 @@ class W3WMapManager(
         listName: String = LIST_DEFAULT_ID,
         zoomOption: W3WZoomOption = W3WZoomOption.CENTER_AND_ZOOM,
     ): List<W3WResult<W3WMarker>> = withContext(dispatcher) {
-        val results = inputs.map { input ->
+        val conversionResults = inputs.map { input ->
             async {
-                // Convert and add marker
-                when (val result = convertFunction(input)) {
-                    is W3WResult.Success -> {
-                        val marker = result.value.toW3WMarker(markerColor)
-                        markersMap.addMarker(listName = listName, marker = marker)
-                    }
+                convertFunction(input)
+            }
+        }.awaitAll()
 
-                    is W3WResult.Failure -> W3WResult.Failure(result.error, result.message)
+        val results = conversionResults.map { result ->
+            when (result) {
+                is W3WResult.Success -> {
+                    val marker = result.value.toW3WMarker(markerColor)
+                    markersMap.addMarker(listName = listName, marker = marker)
                 }
+                is W3WResult.Failure -> W3WResult.Failure(result.error, result.message)
             }
-        }.awaitAll()
+        }
 
         // Update the map state
         _mapState.update { currentState ->