[GitHub] [cordova-android] breautek commented on issue #924: Crash Report: ConcurrentModificationException

[GitBox <gi...@apache.org>]

breautek commented on issue #924:
URL: https://github.com/apache/cordova-android/issues/924#issuecomment-618661142


   Hmm, looks like it, just in a different area. But it's still inside a loop like the original 2 stacktraces. Using `synchronizedMap` had a note that if using `Iterator`, then we need to wrap it in a `synchronized` block.
   
   We don't use iterators directly, we use a for in block... so I didn't think we needed the `synchronized` block.... but I guess behind-the-scenes it still does use `java.util.LinkedHashMap$LinkedValueIterator` so maybe we do need to wrap every `for...in` block inside a `synchronized {}`...
   
   And I trust that your new stacktrace crash came from an updated version of the app that contains the https://github.com/breautek/cordova-android/commit/bfda452a09ba26412e0f527f6af1957379d89453 patch?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org