Don't use intern'ed strings to synchronize on

Description

Openfire uses strings (often namespaced, to reduce the likelyhood of problems) as locks, eg:

Although the namespacing severely reduces the likelyhood of accidental / unintended re-use of the lock in unrelated code (which introduces the potential for deadlocks), this solution is far from ideal. A better one should be found.

Environment

None

Acceptance Test - Entry

None
Fixed

Assignee

Guus der Kinderen

Reporter

Guus der Kinderen

Labels

None

Expected Effort

None

Ignite Forum URL

None

Fix versions

Priority

Major
Configure