- Reduces garbage created when working with AccoutID.getAccountProperties() which creates a copy of the internal Hashtable by introducing getAccountProperty(), getAccountPropertyString(), getAccountPropertyBoolean() and getAccountPropertyInt(). The last two also carry out the conversions between Object and boolean/int without additional allocations in contrast to most of the previous uses which would perform allocations.

- Modifies SimpleStatusMenu to load its necessary image once instead of three times and to spare a ImageIcon instance.

1 files changed, 3 insertions, 3 deletions

diff --git a/src/net/java/sip/communicator/impl/systray/jdic/StatusSubMenu.java b/src/net/java/sip/communicator/impl/systray/jdic/StatusSubMenu.java
index 1d5a96f..5b178c0 100644
--- a/src/net/java/sip/communicator/impl/systray/jdic/StatusSubMenu.java
+++ b/src/net/java/sip/communicator/impl/systray/jdic/StatusSubMenu.java
@@ -184,9 +184,9 @@ public class StatusSubMenu
                     = (ProtocolProviderService) SystrayActivator.bundleContext
                         .getService(protocolProviderRefs[i]);
 
-                boolean isHidden = 
-                    provider.getAccountID().getAccountProperties().
-                        get(ProtocolProviderFactory.IS_PROTOCOL_HIDDEN) != null;
+                boolean isHidden =
+                    provider.getAccountID().getAccountProperty(
+                        ProtocolProviderFactory.IS_PROTOCOL_HIDDEN) != null;
                 
                 if(!isHidden)
                     this.addAccount(provider);