Concurrency fix - Githubissues

eclipse-archived / concierge

Eclipse Concierge™ project

https://www.eclipse.org/concierge/

Eclipse Public License 1.0

33 stars 22 forks source link

Closed tverbele closed 8 years ago

tverbele commented 8 years ago

Two fixes regarding potential concurrency issues:

we take copies of the service listeners arrays, but at one point this was omitted/forgotten - fixed that
properties of a service can be set by multiple threads simultaneously ... add synchronized block around the change

JochenHiller commented 8 years ago

Looks good to me. Would we be able to write tests for that changes? Or too difficult due to concurrency needs?

tverbele commented 8 years ago

Hard to test these concurrecy issues. I fixed them after I witnessed ConcurrentModificationExceptions once in a while...