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]].

Reply via email to