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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
