jmeter-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pmoua...@apache.org
Subject svn commit: r1549031 - in /jmeter/trunk: src/core/org/apache/jmeter/testelement/property/AbstractProperty.java xdocs/changes.xml
Date Sun, 08 Dec 2013 14:43:05 GMT
Author: pmouawad
Date: Sun Dec  8 14:43:05 2013
New Revision: 1549031

URL: http://svn.apache.org/r1549031
Log:
Bug 55857 - Performance : AbstractProperty should test for emptiness to avoid Exception throwing
Bugzilla Id: 55857

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/testelement/property/AbstractProperty.java
    jmeter/trunk/xdocs/changes.xml

Modified: jmeter/trunk/src/core/org/apache/jmeter/testelement/property/AbstractProperty.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/testelement/property/AbstractProperty.java?rev=1549031&r1=1549030&r2=1549031&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/testelement/property/AbstractProperty.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/testelement/property/AbstractProperty.java Sun
Dec  8 14:43:05 2013
@@ -108,7 +108,7 @@ public abstract class AbstractProperty i
     @Override
     public int getIntValue() {
         String val = getStringValue();
-        if (val == null) {
+        if (val == null || val.length()==0) {
             return 0;
         }
         try {
@@ -126,7 +126,7 @@ public abstract class AbstractProperty i
     @Override
     public long getLongValue() {
         String val = getStringValue();
-        if (val == null) {
+        if (val == null || val.length()==0) {
             return 0;
         }
         try {
@@ -144,7 +144,7 @@ public abstract class AbstractProperty i
     @Override
     public double getDoubleValue() {
         String val = getStringValue();
-        if (val == null) {
+        if (val == null || val.length()==0) {
             return 0;
         }
         try {
@@ -163,7 +163,7 @@ public abstract class AbstractProperty i
     @Override
     public float getFloatValue() {
         String val = getStringValue();
-        if (val == null) {
+        if (val == null || val.length()==0) {
             return 0;
         }
         try {
@@ -182,7 +182,7 @@ public abstract class AbstractProperty i
     @Override
     public boolean getBooleanValue() {
         String val = getStringValue();
-        if (val == null) {
+        if (val == null || val.length()==0) {
             return false;
         }
         return Boolean.parseBoolean(val);

Modified: jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1549031&r1=1549030&r2=1549031&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml (original)
+++ jmeter/trunk/xdocs/changes.xml Sun Dec  8 14:43:05 2013
@@ -207,6 +207,7 @@ A workaround is to use a Java 7 update 4
 <li><bugzilla>55753</bugzilla> - Improve FilePanel behaviour to start from
the value set in Filename field if any. Contributed by UBIK Load Pack (support at ubikloadpack.com)</li>
 <li><bugzilla>55756</bugzilla> - HTTP Mirror Server : Add ability to set
Headers</li>
 <li><bugzilla>55852</bugzilla> - Be more lenient in parsing when charset
value is surrounded with single quotes</li>
+<li><bugzilla>55857</bugzilla> - Performance : AbstractProperty should
test for emptiness to avoid Exception throwing</li>
 </ul>
 
 <ch_section>Non-functional changes</ch_section>



Mime
View raw message