svn commit: r19116 - trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java

Author: bobtarling
Date: 2011-03-14 19:40:15-0700
New Revision: 19116

Modified:
   trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java

Log:
Use the handy local variable instead of calling the same class method repeatedly.

Modified: trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java?view=diff&pathrev=19116&r1=19115&r2=19116
==============================================================================
--- trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java	(original)
+++ trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java	2011-03-14 19:40:15-0700
@@ -207,14 +207,14 @@
         // instantiated when its not always needed.
         JPanel p = new JPanel();
         
-        final TitledBorder border = new TitledBorder(prop.getPropertyName());        
-        p.setBorder(border);
-
         JComponent control = null;
         
         final String propertyName = prop.getPropertyName();
         final Class<?> type = prop.getType();
         
+        final TitledBorder border = new TitledBorder(propertyName);        
+        p.setBorder(border);
+
         if ("initialValue".equals(propertyName)) {        
             UMLExpressionModel model = 
                 new UMLInitialValueExpressionModel(target);

------------------------------------------------------
http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=2711525

To unsubscribe from this discussion, e-mail: [[email protected]].