breautek commented on issue #924: URL: https://github.com/apache/cordova-android/issues/924#issuecomment-708813202
> I can confirm that just changing to using the synchronizedMap does not fix the issue. Yah, I provided a test fix for @EinfachHans awhile back that simply changed the lists to a `synchronizedMap` since it sounds like that was all that was required to be thread-safe according the docs, but EinfachHans reported the same result. > I will add that even if PR #1073 works, it will not fix all ConcurrentModificationException, because CME occurs on other methods as well. Glancing over the PR, it looks like you've updated all methods to be synchronized? > Not sure why the PR #1091 failed macOs tests.... I don't think it's the fault of the PR, it appears to be a timeout issue, I can try restarting the tests. ---------------------------------------------------------------- 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]
