Author: sebb
Date: Thu Jun 26 15:41:26 2008
New Revision: 672045

URL: http://svn.apache.org/viewvc?rev=672045&view=rev
Log:
Enable setProperty with default

Modified:
    
jakarta/jmeter/trunk/src/core/org/apache/jmeter/testelement/AbstractTestElement.java

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/testelement/AbstractTestElement.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/testelement/AbstractTestElement.java?rev=672045&r1=672044&r2=672045&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/testelement/AbstractTestElement.java
 (original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/testelement/AbstractTestElement.java
 Thu Jun 26 15:41:26 2008
@@ -265,27 +265,24 @@
                setProperty(new StringProperty(name, value));
        }
 
-// TODO - enable this for the next version
-// Doing so now will generate huge Javadoc changes...
-
-//    /**
-//     * Create a String property - but only if it is not the default.
-//     * This is intended for use when adding new properties to JMeter
-//     * so that JMX files are not expanded unnecessarily.
-//     * 
-//     * N.B. - must agree with the default applied when reading the property.
-//     * 
-//     * @param name property name
-//     * @param value current value
-//     * @param dflt default
-//     */
-//    public void setProperty(String name, String value, String dflt) {
-//        if (dflt.equals(value)) {
-//            removeProperty(name);
-//        } else {
-//            setProperty(new StringProperty(name, value));
-//        }
-//    }
+    /**
+     * Create a String property - but only if it is not the default.
+     * This is intended for use when adding new properties to JMeter
+     * so that JMX files are not expanded unnecessarily.
+     * 
+     * N.B. - must agree with the default applied when reading the property.
+     * 
+     * @param name property name
+     * @param value current value
+     * @param dflt default
+     */
+    public void setProperty(String name, String value, String dflt) {
+        if (dflt.equals(value)) {
+            removeProperty(name);
+        } else {
+            setProperty(new StringProperty(name, value));
+        }
+    }
 
        public void setProperty(String name, boolean value) {
                setProperty(new StringProperty(name, Boolean.toString(value)));



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to