Fixes a couple of occurrences of incorrect code, performance-related warnings.

1 files changed, 13 insertions, 14 deletions

diff --git a/src/net/java/sip/communicator/util/swing/SIPCommFrame.java b/src/net/java/sip/communicator/util/swing/SIPCommFrame.java
index db14b42..07b5311 100644
--- a/src/net/java/sip/communicator/util/swing/SIPCommFrame.java
+++ b/src/net/java/sip/communicator/util/swing/SIPCommFrame.java
@@ -135,22 +135,9 @@ public abstract class SIPCommFrame
      */
     private void saveSizeAndLocation()
     {
-        ConfigurationService configService =
-            UtilActivator.getConfigurationService();
-
-        String className = this.getClass().getName();
-
         try
         {
-            configService.setProperty(className + ".width", new Integer(
-                getWidth()));
-
-            configService.setProperty(className + ".height", new Integer(
-                getHeight()));
-
-            configService.setProperty(className + ".x", new Integer(getX()));
-
-            configService.setProperty(className + ".y", new Integer(getY()));
+            saveSizeAndLocation(this);
         }
         catch (PropertyVetoException e1)
         {
@@ -159,6 +146,18 @@ public abstract class SIPCommFrame
         }
     }
 
+    static void saveSizeAndLocation(Component component)
+        throws PropertyVetoException
+    {
+        ConfigurationService config = UtilActivator.getConfigurationService();
+        String className = component.getClass().getName();
+
+        config.setProperty(className + ".width", component.getWidth());
+        config.setProperty(className + ".height", component.getHeight());
+        config.setProperty(className + ".x", component.getX());
+        config.setProperty(className + ".y", component.getY());
+    }
+
     /**
      * Sets window size and position.
      */