[dmdirc commit] r5811 - trunk/src/com/dmdirc/config

[com...@dmdirc.com]

Author: chris87
Date: 2010-01-19 16:54:04 +0000 (Tue, 19 Jan 2010)
New Revision: 5811

Modified:
trunk/src/com/dmdirc/config/Identity.java
Log:
Identity.unsetOption no longer holds locks when firing callbacks

Git-version: 0.6.3m2-338-ga76a8b7

Modified: trunk/src/com/dmdirc/config/Identity.java
===================================================================
--- trunk/src/com/dmdirc/config/Identity.java 2010-01-19 16:53:05 UTC (rev 5810)
+++ trunk/src/com/dmdirc/config/Identity.java 2010-01-19 16:54:04 UTC (rev 5811)
@@ -433,9 +433,11 @@
* @param domain domain of the option
* @param option name of the option
*/
- public synchronized void unsetOption(final String domain, final String option) {
- file.getKeyDomain(domain).remove(option);
- needSave = true;
+ public void unsetOption(final String domain, final String option) {
+ synchronized (this) {
+ file.getKeyDomain(domain).remove(option);
+ needSave = true;
+ }

fireSettingChange(domain, option);
}