-
Notifications
You must be signed in to change notification settings - Fork 3
Design
HenryHu edited this page Apr 5, 2012
·
2 revisions
Currently, the system is multi-threaded. Since there is no synchronization, there are many problems here.
Notify thread accessed a ServerCore which has not been initialized.
Updater thread was iterating on the rosters._rosters, while a user logged in/logged out, changes rosters._rosters
Either: 1. Add synchronization 2. Restore single-threaded design
Care: deadlock
Allow delegated method execution: still requires synchronization Remove other threads: how to inject timeout events?
Reason unknown. "presence" tag in wrong order?